Bug 461590 - Border line style should be customizable from VSM
Summary: Border line style should be customizable from VSM
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 2.0.0   Edit
Hardware: All All
: P1 enhancement (vote)
Target Milestone: 3.1.0   Edit
Assignee: Maxime Porhel CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks: 472183 474677 436359 470859
  Show dependency tree
 
Reported: 2015-03-06 09:14 EST by Maxime Porhel CLA
Modified: 2015-10-16 09:12 EDT (History)
3 users (show)

See Also:


Attachments
Solid, dot, dash_dot line style (bordersize:3) (532.14 KB, image/png)
2015-08-10 11:30 EDT, Maxime Porhel CLA
no flags Details
Solid, dot, dash_dot line styles (borderSize/edgeSize = 3) (532.13 KB, image/png)
2015-08-10 11:31 EDT, Maxime Porhel CLA
no flags Details
Solid, dot, dash_dot line styles (borderSize/edgeSize = 3) (529.39 KB, image/png)
2015-08-10 11:32 EDT, Maxime Porhel CLA
no flags Details
Dash line style for 1, 3 and 5 pix widths (532.13 KB, image/png)
2015-08-10 11:36 EDT, Maxime Porhel CLA
no flags Details
Line style test case (5.56 KB, application/zip)
2015-08-11 09:04 EDT, Maxime Porhel CLA
no flags Details
Final dash line style for 1, 3 and 5 pix widths (183.23 KB, image/png)
2015-08-19 10:54 EDT, Maxime Porhel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Maxime Porhel CLA 2015-03-06 09:14:15 EST
BordereredStyleDescription and BorderedStyle should have an attribute typed by LineStyle. 

The default value should be solid (as current borders) (no required migration).

This will allow the specifier do choose the border line style for containers and node mappings. 

All our Node and Container figures have to be updated to handle the chosen line style from the VSM (description -> style, style to figure).  The customization from the style property section should put the attribute in the custom list. The figure should be update when the style change (style customization, conditional style, user change from the property sections) (see the line width setter/getter/changes).

The appearance property section might propose to change the line style. 
The tabbar might potentially propose this too.
Comment 1 Maxime Porhel CLA 2015-04-01 05:17:51 EDT
See https://git.eclipse.org/r/#/c/44490/
Comment 2 Maxime Porhel CLA 2015-04-01 05:18:04 EDT
See https://git.eclipse.org/r/#/c/44490/
Comment 3 Maxime Porhel CLA 2015-04-01 05:20:09 EDT
(In reply to Maxime Porhel from comment #0)

> The tabbar might potentially propose this too.

The line style button will not be added to the tabbar: it is currently not available for edges and will be sufficient in the appearance properties tab.
Comment 4 Belqassim Djafer CLA 2015-04-08 04:02:43 EDT
I put work around this ticket paused (see: https://git.eclipse.org/r/#/c/44490/) and here the list of work to do :
1- Add the line style button in the appearance properties tab (the button is grayed and it must be enabled for nodes and containers).
2- Complete the test org.eclipse.sirius.tests.swtbot.BorderLineStyleTest.
3- Update the Release notes.
Comment 5 Eclipse Genie CLA 2015-07-02 05:54:42 EDT
WARNING: this patchset contains 1062 new lines of code and may require a Contribution Questionnaire (CQ) if the author is not a committer on the project. Please see:https://wiki.eclipse.org/Project_Management_Infrastructure/Creating_A_Contribution_Questionnaire
Comment 6 Eclipse Genie CLA 2015-07-02 07:01:54 EDT
New Gerrit change created: https://git.eclipse.org/r/51240
Comment 7 Eclipse Genie CLA 2015-07-02 07:01:56 EDT
New Gerrit change created: https://git.eclipse.org/r/51241
Comment 8 Eclipse Genie CLA 2015-07-02 08:45:02 EDT
New Gerrit change created: https://git.eclipse.org/r/51251
Comment 9 Maxime Porhel CLA 2015-08-10 11:30:22 EDT
Created attachment 255758 [details]
Solid, dot, dash_dot line style (bordersize:3)
Comment 10 Maxime Porhel CLA 2015-08-10 11:31:25 EDT
Created attachment 255759 [details]
Solid, dot, dash_dot line styles (borderSize/edgeSize = 3)
Comment 11 Maxime Porhel CLA 2015-08-10 11:32:53 EDT
Created attachment 255760 [details]
Solid, dot, dash_dot line styles (borderSize/edgeSize = 3)
Comment 12 Maxime Porhel CLA 2015-08-10 11:36:49 EDT
Created attachment 255761 [details]
Dash line style for 1, 3 and 5 pix widths

This show the differences in the dash line style rendering between the different existing styles: this comes from the fact that we use a pattern and the Graphics.LINE_CUSTOM (ie. SWT.LINE_CUSTOM) instead of LINE_DASH when the service is available on the figures in the current patch set as it was done for edges from the beginning of Sirius. 

We might change this: 
 . For the edge, but it will change the rendering of all existing diagrams
 . Keep this pattern only for the edges
 . Extends all used figures to support the LINE_CUSTOM and its dash pattern.
Comment 13 Maxime Porhel CLA 2015-08-10 11:38:42 EDT
For the validation: see org.eclipse.sirius.tests.swtbot.BorderLineStyleTest, the test data has been created with layer and style customizations to easily change the border/edge size and line style of all figures in "one click".
Comment 14 Maxime Porhel CLA 2015-08-11 08:57:47 EDT
BorderLineStyleTest has been renamed into LineStyleTest as it also test the udpate of the edge line style.
Comment 15 Maxime Porhel CLA 2015-08-11 09:04:16 EDT
Created attachment 255784 [details]
Line style test case
Comment 16 Maxime Porhel CLA 2015-08-11 09:09:55 EDT
Steps to reproduce/validate: 
 . Unzip the attached project
 . Open the diagram
 . (De)Activate the layers to change the line style
 . Select each node/container/edge and play with the Style property tab to change the line style / border line style and the size / border size. 
 . Use the reset style to default property to reset the style to the VSM configuration
Comment 23 Maxime Porhel CLA 2015-08-19 10:15:17 EDT
The dash pattern {5, 5} will only be used for edges. 
The node and containers will use the standard SWT.LINE_DASH line style
Comment 24 Eclipse Genie CLA 2015-08-19 10:41:42 EDT
New Gerrit change created: https://git.eclipse.org/r/54116
Comment 25 Maxime Porhel CLA 2015-08-19 10:54:58 EDT
Created attachment 255955 [details]
Final dash line style for 1, 3 and 5 pix widths
Comment 27 Laurent Fasani CLA 2015-10-08 09:42:30 EDT
validated with solid, dash, dash_dot and dash_dot_dot styles.
Double style is excluded from this ticket
Comment 28 Pierre-Charles David CLA 2015-10-16 09:12:34 EDT
Available in Sirius 3.1.0.