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

Bug 465952

Summary: Improve selection of deeply-contained VSM elements
Product: [Modeling] Sirius Reporter: Steve Monnier <steve.monnier>
Component: DiagramAssignee: Pierre-Charles David <pierre-charles.david>
Status: CLOSED FIXED QA Contact: Julien Dupont <julien.dupont>
Severity: major    
Priority: P2 CC: esteban.dugueperoux, julien.dupont, laurent.redor, pierre-charles.david, vincent.richard
Version: 2.0.4Keywords: triaged
Target Milestone: 4.1.0   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/46895
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=db2ca98f284d3ccbada371ad9ae258d184d0ece7
https://git.eclipse.org/r/79693
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=0e4e25404f21d26012f2598f098570b130fb758b
https://git.eclipse.org/r/83037
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=d7b9bc29b1f3bfba6c6a05ad7ad7579b372e8428
Whiteboard:
Bug Depends on:    
Bug Blocks: 500105    
Attachments:
Description Flags
Style customisation's style selection dialog box presenting the styles in an undistinguishable way. none

Description Steve Monnier CLA 2015-04-30 10:42:20 EDT
In a style property customization, the "applied on" field opens a popup with every styles of the VSM. As they are identified with the type and the color (e.g. square blue) a lot of entries are alike and it is hard to select the good one.
It would be more user friendly if the mapping container of the style was also presented. It could be either by adding the id of the mapping before the style label, or presenting the view as a tree of mapping with styles as sub-elements.
Comment 1 Eclipse Genie CLA 2015-04-30 11:01:50 EDT
New Gerrit change created: https://git.eclipse.org/r/46895
Comment 2 Steve Monnier CLA 2015-04-30 11:02:40 EDT
Note that the previous gerrit is a draft of a solution where the id of the container mapping is displayed before the style label.
Comment 3 Esteban DUGUEPEROUX CLA 2015-06-26 08:37:18 EDT
It is a regression as with the release 6.8 of Sirius before it was open-source, the full qualified name of style was displayed. I change the importance of this bug as it make style customization very hard to do when having several style.
Comment 4 Esteban DUGUEPEROUX CLA 2015-06-29 11:24:10 EDT
This regression occurs since diagram metamodel split in its own .ecore, i.e. diagram.ecore. 
Because org.eclipse.sirius.diagram.ui.business.internal.dialect.HierarchyLabelProvider.isClassOfDescriptionPackageOrSubPackage() check.
Comment 5 Vincent Richard CLA 2016-01-12 05:45:21 EST
Created attachment 259121 [details]
Style customisation's style selection dialog box presenting the styles in an undistinguishable way.
Comment 6 Vincent Richard CLA 2016-01-12 05:45:46 EST
When referencing a style in a style customisation the selection dialog box presents the styles in an undistinguishable way (see screenshot).

A workaround for the gradient style is to use named custom colors for the background and foreground colors to distinguish them. But cases occurs where the styles distinguish in other ways than the foreground and background colors (it could be the label expression, the lable color, etc.)

If the styles were named, it would be easy to find the correct one. At least, the mapping's name containing the style could pe displayed to help improve the customer expericence.
Comment 7 Pierre-Charles David CLA 2016-05-11 07:06:21 EDT
Too late for 4.0, but will probably be done in 4.1 (to be released around november). Depending on the impacts, it may be backported to a 4.0.x maintenance release.
Comment 8 Pierre-Charles David CLA 2016-08-19 03:54:01 EDT
Taking this, I've updated and reworked a little the existing patch.
Comment 10 Pierre-Charles David CLA 2016-08-24 04:43:35 EDT
Fixed, but still needs automated tests.
Comment 11 Eclipse Genie CLA 2016-08-25 03:17:37 EDT
New Gerrit change created: https://git.eclipse.org/r/79693
Comment 13 Pierre-Charles David CLA 2016-10-04 08:42:02 EDT
Fixed.
Comment 14 Eclipse Genie CLA 2016-10-12 09:58:55 EDT
New Gerrit change created: https://git.eclipse.org/r/83037
Comment 16 Pierre-Charles David CLA 2016-10-18 11:07:23 EDT
Available in Sirius 4.1.0, see https://wiki.eclipse.org/Sirius/4.1.0 for details.