Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 318499

Summary: CTabFolder rendering problems
Product: [Eclipse Project] e4 Reporter: Boris Bokowski <bokowski>
Component: UIAssignee: Bogdan Gheorghe <gheorghe>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: emoffatt, gheorghe, pwebster, remy.suen, susan
Version: unspecified   
Target Milestone: 1.0 RC2   
Hardware: PC   
OS: All   
Whiteboard:
Attachments:
Description Flags
screenshot for problem 1 and 2
none
screenshot for problem 3
none
Patch
none
More changes to above patch
none
Safe up the get/set TopRight code
none
screenshot with white toolbar
none
screenshot with white composite next to tabs
none
Take 3
none
Add/remove the resize listener
none
Add/remove the resize listener - safer none

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. ***