This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 426424 - [Trim] Polish new min/max alternative
Summary: [Trim] Polish new min/max alternative
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.4   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 4.4 M7   Edit
Assignee: Eric Moffatt CLA
QA Contact: Eric Moffatt CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-23 03:31 EST by Dani Megert CLA
Modified: 2014-04-29 10:59 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2014-01-23 03:31:41 EST
Bug 424489 introduced an alternative to the existing min/max concept.

This bug captures the ongoing discussions and improvements for this feature.
Comment 1 Dani Megert CLA 2014-01-23 03:32:58 EST
From Markus Keller, bug 424489 comment #17:

- Toggling "Use Overlays" doesn't toggle an already-opened fast view.

- It's a bit strange that the "Use Overlays" popup menu is only on the "Restore" button. The setting doesn't affect the "Restore" button. It affects the behavior of the hidden parts buttons, and it makes their "Orientation" popup menu choices pointless.
Comment 2 Dani Megert CLA 2014-01-23 03:34:28 EST
> - It's a bit strange that the "Use Overlays" popup menu is only on the
> "Restore" button. The setting doesn't affect the "Restore" button. It
> affects the behavior of the hidden parts buttons, and it makes their
> "Orientation" popup menu choices pointless.

We could put the menu item into the menu of each view icon, and synchronize the state for now. Maybe later, we could even set this per view? This should also make it easier to remove the 'Orientation' menu when in the new mode.

Also, "Overlays" sounds a bit technical. How about:
Show at Original Location


I think some people might like this new mode, but I'm not sure whether making it the default is good though. Problems I encountered when using it:
- The iew no longer open close to the icon. In cases where I longer know 
  the original place, it looks like it opens at some random place.
- The workbench relayouts to make place for the stack. Sometimes this is quite
  noisy and heavyweight, especially when one only wants to quickly glance at
  a view (that's why we called them fast views ;-).
Comment 3 Dani Megert CLA 2014-01-23 05:59:40 EST
Moving back to Eric. I think it makes sense if he finishes this.
Comment 4 Eric Moffatt CLA 2014-02-14 15:21:32 EST
Thanks guys, this is indeed good input. I really like the idea of eventually being able to make this per view since only a small subset of views get shown automatically and obscure the editor (Console, Debug, Search...).

BTW, I've been using it for some time and while it's certainly OK for most things you can really get hosed by the new mode:

1) Maximize an editor in the Debug perspective
2) activate the Debug view (with Use Overlays off) 
3) Now try to double-click to place a break point

What happens is that the first click activates the editor -> the Debug view gets torn down -> the layout changes and we end up on the wrong lines of code !
Comment 5 Eric Moffatt CLA 2014-04-11 14:04:02 EDT
Committed:

http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=fc0154a1369c6824aa0bcc8a324ba93d3462ea8e

This changes the menu item's text to 'Show in the original location' as well as making the show 'mode' stack based.

I'm going to continue to work some more on this to see if I can arrange that the MINIMIZED_BY_ZOOM state can be maintained across a 'Restore'. This would allow users to actually restore the stack for as long as they want then minimize it again *without* having the stack not auto-restore on the unmax as it currently does...
Comment 6 Eric Moffatt CLA 2014-04-11 14:18:51 EDT
Committed:

http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=7c113f7d02a9990ade430399112e7b39b2d394c9

This maintains the 'MINIMIZED_BY_ZOOM' state until an actual 'unzoom' is performed, regardless of whether the stack is restored in the interim.
Comment 7 Eric Moffatt CLA 2014-04-29 10:59:02 EDT
Verified in 4.4.0.I20140428-2000.