Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 315290 - Implement e4 minimize/maximize behavior
Summary: Implement e4 minimize/maximize behavior
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 1.0 RC1   Edit
Assignee: Project Inbox CLA
QA Contact: Eric Moffatt CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-01 16:07 EDT by Eric Moffatt CLA
Modified: 2010-07-24 17:54 EDT (History)
2 users (show)

See Also:


Attachments
Part 1: Minimize a PartStack (40.65 KB, patch)
2010-06-01 16:13 EDT, Eric Moffatt CLA
no flags Details | Diff
Patch to implement minimize behavior (48.36 KB, patch)
2010-06-03 14:36 EDT, Eric Moffatt CLA
no flags Details | Diff
Patch that puts minimized stacks only in the left/rigth trims areas (22.22 KB, patch)
2010-06-14 10:52 EDT, Eric Moffatt CLA
no flags Details | Diff
Patch to fix an issue on the Mac (1.17 KB, patch)
2010-06-17 12:47 EDT, Eric Moffatt CLA
no flags Details | Diff
Allow toggling the stack by clicking on the toolbar button (1.50 KB, patch)
2010-06-18 09:59 EDT, Eric Moffatt CLA
no flags Details | Diff
Patch to fix restart errors (4.55 KB, patch)
2010-06-18 15:09 EDT, Eric Moffatt CLA
no flags Details | Diff
Switch over to using a Composite wirh custom sizing behavior (22.37 KB, patch)
2010-06-24 13:56 EDT, Eric Moffatt CLA
no flags Details | Diff
Keep any showing hostPane synch'd with window changes (2.59 KB, patch)
2010-06-24 14:27 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 Eric Moffatt CLA 2010-06-01 16:07:10 EDT
We need to (re)implement the current 3.x mechanism for minimizing a PartStack.
Comment 1 Eric Moffatt CLA 2010-06-01 16:13:47 EDT
Created attachment 170694 [details]
Part 1: Minimize a PartStack


This is definitely a work in progress but this patch allows you to minimize a ParStack.

What's in:

Minimize a stack..goes to the TOP trim only for
Allows click on an item to show the stack
bringToTop shows the stack
Switching perspectives will tear down any showing minimized stack and also show/hide stacks based on the perspective.

To close the stack you currently have to click outside its bounds but inside the MWindow.

What's not in:

Sending the TrimStack to the correct 'side'
Updating the TrimStack when views come/go/update state
No DnD, either for views within the TrimStack or the trim control itself.
Comment 2 Eric Moffatt CLA 2010-06-01 16:16:06 EDT
Remy, if you could kick this around a bit I'd really appreciate it. I'm more interested if you can get into a 'broken' state rather than polish issues at this point (I know there are a few...at least...;-).
Comment 3 Remy Suen CLA 2010-06-02 09:06:05 EDT
(In reply to comment #2)
> Remy, if you could kick this around a bit I'd really appreciate it. I'm more
> interested if you can get into a 'broken' state rather than polish issues at
> this point (I know there are a few...at least...;-).

Eric, I tried the patch a bit last night on my Windows 7 machine. I didn't see any obvious problems like exceptions being logged to the console. It seems kind of annoying that clicking one view in one stack and then another causes the shell to go down and back up even though the shell's showing the whole stack but I guess that's technically the same behaviour that happens on 3.x minus the display of the entire stack.
Comment 4 Eric Moffatt CLA 2010-06-02 09:17:56 EDT
Thanks Remy, that is just one of the polish issues I'll try to address today. 

One thing that I don't know how to solve is 'giving up' the active part when I close the minimized stack. Right now I just leave whatever part was active alone but it should really go back to some view/editor visible in the presentation.
Comment 5 Eric Moffatt CLA 2010-06-03 14:36:42 EDT
Created attachment 171000 [details]
Patch to implement minimize behavior


Seems pretty complete except that we need a way to cause the PartService to select a new active part once a 'fast stack' is made invisible...
Comment 6 Eric Moffatt CLA 2010-06-14 10:52:21 EDT
Created attachment 171832 [details]
Patch that puts minimized stacks only in the left/rigth trims areas
Comment 7 Eric Moffatt CLA 2010-06-14 10:53:53 EDT
Committed in >20100614. Applied the patch.
Comment 8 Eric Moffatt CLA 2010-06-17 12:47:04 EDT
Created attachment 172141 [details]
Patch to fix an issue on the Mac


Had to slightly revamp the code to allow for a mac defect (Bogdan's opening that one).
Comment 9 Eric Moffatt CLA 2010-06-17 12:48:14 EDT
Committed in >20100617. Applied the patch.
Comment 10 Eric Moffatt CLA 2010-06-18 09:59:52 EDT
Created attachment 172219 [details]
Allow toggling the stack by clicking on the toolbar button


Clicking on the 'selected' view in a minimized stack that is currently visible should tear it down.
Comment 11 Eric Moffatt CLA 2010-06-18 10:01:54 EDT
Committed in >20100618. Applied the patch.
Comment 12 Eric Moffatt CLA 2010-06-18 15:09:14 EDT
Created attachment 172247 [details]
Patch to fix restart errors


Under certain conditions the minimized stacks were not appearing on a restart.
Comment 13 Eric Moffatt CLA 2010-06-21 13:00:21 EDT
Marking for work in RC1.
Comment 14 Eric Moffatt CLA 2010-06-24 13:56:33 EDT
Created attachment 172657 [details]
Switch over to using a Composite wirh custom sizing behavior
Comment 15 Eric Moffatt CLA 2010-06-24 13:58:45 EDT
Committed in >20100624. Applied the patch.

There's still some work to be done on:

Keeping the hostPane in synch with size changes to the main shell
Handling closing the last view
Handle dragging the last view out of the stack
Comment 16 Eric Moffatt CLA 2010-06-24 14:27:38 EDT
Created attachment 172659 [details]
Keep any showing hostPane synch'd with window changes
Comment 17 Eric Moffatt CLA 2010-06-24 14:28:29 EDT
Committed in >20100624. Applied the patch.
Comment 18 Boris Bokowski CLA 2010-07-24 17:54:37 EDT
(In reply to comment #17)
> Committed in >20100624. Applied the patch.