This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 409340 - [e4] Add support for min/max
Summary: [e4] Add support for min/max
Status: RESOLVED FIXED
Alias: None
Product: Efxclipse
Classification: Technology
Component: Runtime (show other bugs)
Version: unspecified   Edit
Hardware: Macintosh Mac OS X
: P3 normal (vote)
Target Milestone: 2.2.0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-28 16:56 EDT by Thomas Schindl CLA
Modified: 2017-03-27 02:31 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Schindl CLA 2013-05-28 16:56:47 EDT
We should add similar min/max support like e4 on SWT
Comment 1 Thomas Schindl CLA 2014-07-17 11:09:05 EDT
retarget for 1.1.0
Comment 2 Christoph Keimel CLA 2015-08-05 08:30:19 EDT
On how Min/Max should work:

I think we actualy only need the maximize action (in regular mode), because that is usualy what a user wants: More space for a specific view for a certain amount of time. The rest of the stacks should minimize automatically. As soon as you restore the maximized stack or one of the minimzed stacks, all of them return to regular mode. So, compared to SWT, let's get rid of the minimize action.

I like the possibility in SWT to active a minimized part to show its stack in a pop-up window temporarily.
Comment 3 Eclipse Genie CLA 2015-11-13 11:37:07 EST
New Gerrit change created: https://git.eclipse.org/r/60347
Comment 5 Thomas Schindl CLA 2015-11-13 13:35:17 EST
I've merged the current version but I'd like you to make follow up changes:
* use @Component annotations (we use https://github.com/ecliptical/ds-annotations) to generate the component.xml
* Add @since 2.2.0 to the interfaces
* I'm like you not a native speaker but unMaximize is certainly not english (I'll follow up on this point)
* I don't think we should contribute the MaximizationServiceContextFunction through DS but through an Addon (MinMaxAddon) this allows people to turn it on/off
* if there's a MaximizationService we should
  - maximize/minimize on double click on the TabItem
  - maximize/minimize on double click on the Parts TopTimArea (see WPart.SHOW_TOP_TRIM_AREA_TAG)
* we should allow the user to contribute a service to veto min/max for a certain MUIElement
* we should make window also a WMaximizationHost because one might develop apps without perspectives

But this is a fantastic step forwards - I highly appreciate the changes!
Comment 6 Thomas Schindl CLA 2015-11-13 13:40:22 EST
On the names:

MaximizationService#unMaximize => MaximizationService#restore
MaximizationTransitionService#unMaximize => MaximizationTransitionService#restore
Comment 7 Thomas Schindl CLA 2015-11-26 05:51:16 EST
I made the following changes:
* I renamed the unmaximize to restore
* added @since
* remove the contribution of the transition service - to align it it with other transitions - if a user wants animation its his responsibility to contribute a service eg by using the default one we provide

More changes to come
Comment 8 Thomas Schindl CLA 2015-11-26 07:26:57 EST
More modifications:
* I left the service and context function as is
* I added a new MinMaxAddon who interacts with the model and widgets and added support for min-max when double clicking on a Tab in the MStackContainer
Comment 9 Thomas Schindl CLA 2015-11-26 07:27:35 EST
* and i move the maximize service and CF to the base bundle because they are not widget toolkit specific
Comment 10 Thomas Schindl CLA 2015-11-26 08:16:57 EST
Argh - Sun was not following this bug!

Final bits implemented:
* add support for applications without perspective

Thanks Sun for this wonderful new feature! If we want to add min/max for none MStack-Elements we should open a new bug to track that.
Comment 11 Eclipse Genie CLA 2015-11-26 16:09:22 EST
New Gerrit change created: https://git.eclipse.org/r/61391
Comment 12 Sun Volland CLA 2015-11-26 16:20:57 EST
(In reply to Eclipse Genie from comment #11)
> New Gerrit change created: https://git.eclipse.org/r/61391

I had some time this evening but just saw you did all the work. Cool to have all this for 2.2.0.
Just added a little contribution to get rid of the context function, can you confirm this is the way to do this (creating the max. service in the minmaxaddon at widget creation time) ?
Now I'm also better understanding the min max callback's purpose.
Thanks a lot too, Tom !
Comment 13 Thomas Schindl CLA 2015-11-26 16:26:04 EST
I changed my mind and think the service is ok this way so i abandoned the change - i hope i haven't wasted much of your time!
Comment 14 Sun Volland CLA 2015-11-26 16:36:18 EST
not at all, learning is never wasted time ;)
Comment 15 Henri Boucher CLA 2017-03-26 15:23:36 EDT
Can I get explicit instructions on where to find and how to install the minmax addon as well as any "Tags" or "Persistent State" that might be required.
Comment 16 Thomas Schindl CLA 2017-03-27 02:31:31 EDT
Please ask usability question at the forum - http://www.eclipse.org/forums/index.php/f/259/