Bug 461593 - Containers should have the hide label capabilities
Summary: Containers should have the hide label capabilities
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 2.0.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0.0   Edit
Assignee: Mickael LANOE CLA
QA Contact:
URL:
Whiteboard: needtest
Keywords: triaged
Depends on: 463765
Blocks: 436359
  Show dependency tree
 
Reported: 2015-03-06 09:29 EST by Maxime Porhel CLA
Modified: 2015-06-24 11:13 EDT (History)
2 users (show)

See Also:


Attachments

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:29:35 EST
The hide label / hide label by default capabilities are not available on lists and containers. 

The "hide label by default" is currently defined on NodeStyleDescription and NodeStyle. It might be extracted in a specific ECLass (interface) then this interface could be added to NodeStyleDescription and ContainerStyleDescription super types.

Tabbar (hide label actions and show/hide wizard), outline will have to be modified. 

For Lists, as there is currently no possibility to change the label area style (there is always a border), the behavior has to be checked when we hide the label. We might choose to keep the hide label item from the tabbar (and from the wizard) disabled for list or to force the label area to collapse when the label is hidden.
Comment 1 Maxime Porhel CLA 2015-03-06 09:37:04 EST
This is a first step to add the capability to define containers and lists without labels. 

After the implementation of the hide label / hide label per default capabilities, a new Bugzilla might be created to handle remaining issues or needed modifications to allow the specifier to describe containers and lists without label.

We have to analyse the result of the several possible combination between  {empty label expression + showIcon:false}, "hide label per default" and the use of the hide label action per the user.
Comment 2 Maxime Porhel CLA 2015-03-06 09:47:28 EST
Note that LABEL_EXPRESSION_EDEFAULT = "feature:name" for LabelStyleDescription, so there is no possibility (except with a service maybe) to define an empty label.
Comment 3 Eclipse Genie CLA 2015-04-01 05:15:04 EDT
New Gerrit change created: https://git.eclipse.org/r/44982
Comment 4 Eclipse Genie CLA 2015-04-01 12:01:36 EDT
New Gerrit change created: https://git.eclipse.org/r/45031
Comment 5 Eclipse Genie CLA 2015-04-01 12:01:38 EDT
New Gerrit change created: https://git.eclipse.org/r/45030
Comment 6 Eclipse Genie CLA 2015-04-01 12:01:39 EDT
New Gerrit change created: https://git.eclipse.org/r/45033
Comment 7 Eclipse Genie CLA 2015-04-01 12:01:40 EDT
New Gerrit change created: https://git.eclipse.org/r/45032
Comment 8 Eclipse Genie CLA 2015-04-01 12:01:42 EDT
New Gerrit change created: https://git.eclipse.org/r/45034
Comment 9 Eclipse Genie CLA 2015-04-02 04:32:05 EDT
New Gerrit change created: https://git.eclipse.org/r/45107
Comment 10 Eclipse Genie CLA 2015-04-02 10:51:11 EDT
New Gerrit change created: https://git.eclipse.org/r/45152
Comment 12 Eclipse Genie CLA 2015-04-07 09:08:34 EDT
New Gerrit change created: https://git.eclipse.org/r/45374
Comment 13 Eclipse Genie CLA 2015-04-07 09:08:36 EDT
New Gerrit change created: https://git.eclipse.org/r/45373

WARNING: this patchset contains 2579 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 14 Eclipse Genie CLA 2015-04-29 09:32:28 EDT
New Gerrit change created: https://git.eclipse.org/r/46779
Comment 22 Eclipse Genie CLA 2015-05-11 05:08:23 EDT
New Gerrit change created: https://git.eclipse.org/r/47607
Comment 24 Maxime Porhel CLA 2015-05-29 03:09:14 EDT
Tests will be done after the release. 
Needtest label is present.
Comment 25 Eclipse Genie CLA 2015-05-29 07:20:30 EDT
New Gerrit change created: https://git.eclipse.org/r/48942
Comment 27 Pierre-Charles David CLA 2015-06-24 11:13:51 EDT
Available in Sirius 3.0.0. See https://wiki.eclipse.org/Sirius/3.0.0.