This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 281771 - Part toolbar does not "move down" if there is insufficient space
Summary: Part toolbar does not "move down" if there is insufficient space
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 0.9   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.0 RC2   Edit
Assignee: Project Inbox CLA
QA Contact: Eric Moffatt CLA
URL:
Whiteboard:
Keywords:
: 317456 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-06-28 08:39 EDT by Remy Suen CLA
Modified: 2010-07-19 16:04 EDT (History)
6 users (show)

See Also:


Attachments
screenshot (5.84 KB, image/png)
2010-07-07 17:58 EDT, Susan McCourt CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Remy Suen CLA 2009-06-28 08:39:07 EDT
1. Open the 'Outline' view (it should be open already).
2. Open the 'Ant' view. It should open in the same stack as the 'Outline' view.
3. Notice that the toolbar has completely eaten up the stack's tabs.
4. Increase the size of that stack horizontally and the tabs will show up.
Comment 1 Eric Moffatt CLA 2009-06-29 09:30:32 EDT
This is the result of not having (yet) duplicated the mechanism used in the 3.x stream to do this. There the PaneFolder is actually managing two controls; 

- a CTabFolder: manages the tabs and min/max affordance and the toolbar if there's enough space)
- a ViewForm: manages the 'message area' and the Toolbar if it won't fit into CTabFolder's top-right area.

This is a somewhat complicated dance between the two controls. Since they're both 'custom' perhaps we can provide a new custom control that combines the capabilities of both (i.e. create a subclass of CTabFolder that can manage the message area and have complete control of the Toolbar location).
Comment 2 Boris Bokowski CLA 2009-06-29 14:50:35 EDT
(In reply to comment #1)
> This is a somewhat complicated dance between the two controls. Since they're
> both 'custom' perhaps we can provide a new custom control that combines the
> capabilities of both (i.e. create a subclass of CTabFolder that can manage the
> message area and have complete control of the Toolbar location).

+1
Comment 3 Boris Bokowski CLA 2009-06-29 14:51:18 EDT
Maybe someone from the SWT team can help us with this?
Comment 4 Eric Moffatt CLA 2010-06-21 14:03:49 EDT
Interestingly this hasn't been reported by anyone as a major defect. At this point trying to provide an implementation for this that matches 3.x would likely jeopardize the release date...
Comment 5 Eric Moffatt CLA 2010-06-21 14:04:51 EDT
Note that it's one of the first issues I think we should address since it can really affect the usability of the view stacks.
Comment 6 Paul Webster CLA 2010-06-21 14:08:45 EDT
While we may have to drop this, I think it should be considered for 4.0.  Almost any use of the same stack that has the package explorer renders the view tabs in that stack unreadable.

PW
Comment 7 Oleg Besedin CLA 2010-06-21 14:28:13 EDT
*** Bug 317456 has been marked as a duplicate of this bug. ***
Comment 8 Oleg Besedin CLA 2010-06-21 14:29:44 EDT
(In reply to comment #4)
> Interestingly this hasn't been reported by anyone as a major defect.

Now it is: bug 317456 :-).

A agree with Paul, while not a ship stopper, this is a major eyesore.
Comment 9 Eric Moffatt CLA 2010-06-22 14:56:13 EDT
Yep, this is the result of the legacy code using a mashed together combination of a CTabItem's control actually being a ViewForm and a whole bunch of wiring...

My sincere hope is that we can address this by augmenting the CTF's behavior to provide this functionality (as well as the 'message area'). Note that we couldn't do this because we couldn't change the CTF's API for 3.6.
Comment 10 Susan McCourt CLA 2010-07-07 17:49:44 EDT
(In reply to comment #4)
> Interestingly this hasn't been reported by anyone as a major defect. At this
> point trying to provide an implementation for this that matches 3.x would
> likely jeopardize the release date...

This is also the subject of bug 314159.  I didn't mark it a duplicate because it has some screenshots that show problematic scenarios and has a higher sev (it was on Dani's stop ship list as bug 318868).

For views with lots of toolbars (such as synchronize) it nearly guarantees that you won't see a view title.
Comment 11 Susan McCourt CLA 2010-07-07 17:58:47 EDT
Created attachment 173715 [details]
screenshot

this screenshot shows how the toolbar itself can seemlingly "dangle" past the left border of the view stack.
Comment 12 Susan McCourt CLA 2010-07-19 16:04:58 EDT
This is fixed by Bogdan's CTabFolder changes.  We can open new bugs for any remaining bugs/cheese/problems with the algorithm.

> this screenshot shows how the toolbar itself can seemlingly "dangle" past the
> left border of the view stack.

This problem still exists and is now the focus of bug 314159