Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 318499 - CTabFolder rendering problems
Summary: CTabFolder rendering problems
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 1.0 RC2   Edit
Assignee: Bogdan Gheorghe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 315086 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-06-30 12:30 EDT by Boris Bokowski CLA
Modified: 2010-07-23 17:05 EDT (History)
5 users (show)

See Also:


Attachments
screenshot for problem 1 and 2 (20.02 KB, image/png)
2010-06-30 12:33 EDT, Boris Bokowski CLA
no flags Details
screenshot for problem 3 (78.50 KB, image/png)
2010-06-30 12:33 EDT, Boris Bokowski CLA
no flags Details
Patch (279.53 KB, patch)
2010-07-15 11:36 EDT, Bogdan Gheorghe CLA
no flags Details | Diff
More changes to above patch (17.13 KB, patch)
2010-07-15 14:14 EDT, Bogdan Gheorghe CLA
no flags Details | Diff
Safe up the get/set TopRight code (1.20 KB, patch)
2010-07-15 16:28 EDT, Eric Moffatt CLA
no flags Details | Diff
screenshot with white toolbar (9.83 KB, image/png)
2010-07-16 11:38 EDT, Boris Bokowski CLA
no flags Details
screenshot with white composite next to tabs (9.65 KB, image/png)
2010-07-16 11:40 EDT, Boris Bokowski CLA
no flags Details
Take 3 (20.34 KB, patch)
2010-07-16 13:44 EDT, Bogdan Gheorghe CLA
no flags Details | Diff
Add/remove the resize listener (1.48 KB, patch)
2010-07-16 15:44 EDT, Eric Moffatt CLA
no flags Details | Diff
Add/remove the resize listener - safer (1.77 KB, patch)
2010-07-16 15:46 EDT, Eric Moffatt CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Boris Bokowski CLA 2010-06-30 12:30:40 EDT
1. The close button is sometimes outside the tab
2. The toolbar can make the tab too small
3. If you open a lot of editors, sometimes the tab for the active editor is not shown at all.

I'll attach screenshots.
Comment 1 Boris Bokowski CLA 2010-06-30 12:33:28 EDT
Created attachment 173124 [details]
screenshot for problem 1 and 2
Comment 2 Boris Bokowski CLA 2010-06-30 12:33:47 EDT
Created attachment 173125 [details]
screenshot for problem 3
Comment 3 Remy Suen CLA 2010-06-30 13:17:54 EDT
(In reply to comment #2)
> Created an attachment (id=173125) [details]
> screenshot for problem 3

There is bug 315086 for this. I see it once in a while.
Comment 4 Susan McCourt CLA 2010-07-07 18:13:07 EDT
see also bug 314159 and bug 281771.
It seems to me that we are wrestling with two separate issues:

- the toolbar is not moving downward (bug 281771) so in some view stacks, you rarely have enough room for a tab.
- the shrinking tab algorithm needs work - (dangling stuff, missing tabs, etc.) 

If we can address the first issue then the second one isn't hit so often.
I talked to Bogdan about this.  He's looking into some possible resize listener hacks that could help.
Barring that, I could investigate some changes in the renderer to get the toolbar moved down?  I completely understand Eric's point in bug 281771 that getting 3.6 behavior is risky at this point, but I'm wondering if there is some more surgical hack to force the toolbar down that we could try (in fact, having the view toolbar always appear below the tabs would be a better solution in my opinion than having the current behavior.)
Comment 5 Boris Bokowski CLA 2010-07-07 18:53:27 EDT
(In reply to comment #4)
> (in fact, having
> the view toolbar always appear below the tabs would be a better solution in my
> opinion than having the current behavior.)

I agree. Maybe this is what we should try to implement first?
Comment 6 Bogdan Gheorghe CLA 2010-07-15 11:36:52 EDT
Created attachment 174411 [details]
Patch
Comment 7 Bogdan Gheorghe CLA 2010-07-15 14:14:03 EDT
Created attachment 174429 [details]
More changes to above patch

Second part of patch
Comment 8 Eric Moffatt CLA 2010-07-15 16:28:10 EDT
Created attachment 174444 [details]
Safe up the get/set TopRight code


The current 4.0 implementation shares the same TB amongst different CTF's. This means that it can arbitrarily reparent the topRightControl, leaving a topRightComposite that no longer has any children.
Comment 9 Eric Moffatt CLA 2010-07-15 16:29:10 EDT
Committed in >20100715. Applied the patch.
Comment 10 Boris Bokowski CLA 2010-07-16 11:38:54 EDT
Created attachment 174508 [details]
screenshot with white toolbar

Bogdan, on Cocoa the new code sometimes produces white background in the toolbar.
Comment 11 Boris Bokowski CLA 2010-07-16 11:40:15 EDT
Created attachment 174509 [details]
screenshot with white composite next to tabs

Perhaps the same problem as the previous screenshot, but the white stays even if the toolbar moves down.
Comment 12 Bogdan Gheorghe CLA 2010-07-16 13:44:07 EDT
Created attachment 174521 [details]
Take 3

Tried another approach that is cleaner...
Comment 13 Bogdan Gheorghe CLA 2010-07-16 14:37:24 EDT
Fixed in HEAD > 20100716
Comment 14 Eric Moffatt CLA 2010-07-16 15:44:10 EDT
Created attachment 174533 [details]
Add/remove the resize listener
Comment 15 Eric Moffatt CLA 2010-07-16 15:46:09 EDT
Created attachment 174534 [details]
Add/remove the resize listener - safer
Comment 16 Eric Moffatt CLA 2010-07-16 15:47:20 EDT
Committed in >20100716. Applied the patch.
Comment 17 Susan McCourt CLA 2010-07-19 16:07:40 EDT
we can use bug 314159 to track the remaining issue - where a close box or chevron can dangle outside the tab.  (The 3.6 approach is prevent the part from getting that thin)...
Comment 18 Boris Bokowski CLA 2010-07-23 17:05:28 EDT
*** Bug 315086 has been marked as a duplicate of this bug. ***