Bug 460966 - [Appearance] Inappropriate compartment properties exposed to users
Summary: [Appearance] Inappropriate compartment properties exposed to users
Status: NEW
Alias: None
Product: Papyrus
Classification: Modeling
Component: Diagram (show other bugs)
Version: 1.1.0   Edit
Hardware: PC Mac OS X
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-26 11:39 EST by Christian Damus CLA
Modified: 2017-09-08 09:46 EDT (History)
1 user (show)

See Also:


Attachments
What I see (138.87 KB, image/png)
2015-02-26 11:39 EST, Christian Damus CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Damus CLA 2015-02-26 11:39:56 EST
Created attachment 251145 [details]
What I see

Mars nightly build Feb 26

The attachment shows what happens when a list compartment is selected and its properties presented in the Appearance tab.

* the Collapsed property is repeated several times
* one of the Collapsed properties is not editable
* the Mutable property is for internal use only (I'm not sure that anything other
  than CanonicalEditPolicy should touch that)
* the Element reference for compartments usually is inferred from the containing shape
  but I can't think of any case where it should ever be modifiable by the user
  (expect diagram corruption)
* the only way source and target connections should be edited is in the diagram.
  And presenting them in lists at all is pointless because the notation views aren't
  interesting as such
Comment 1 Camille Letavernier CLA 2015-02-26 11:43:40 EST
In general, the Appearance properties view should be fixed for many elements. It has been generated, and there are lots of duplicates/useless properties. I think only simple Shapes and Connectors have a proper properties view

However, it should not be possible to select a Compartment in Papyrus (But I guess you need to restore this ability for the synchronization mechanism?)
Comment 2 Christian Damus CLA 2015-02-26 11:50:51 EST
(In reply to Camille Letavernier from comment #1)
> 
> However, it should not be possible to select a Compartment in Papyrus (But I
> guess you need to restore this ability for the synchronization mechanism?)

Interesting.  I did nothing that I know of to make list compartments selectable.  At least, not for synchronization.  Perhaps it was necessary for the pop-up bar assistant (my memory is so short).
Comment 3 Christian Damus CLA 2015-02-26 14:59:10 EST
To be clear, it is not all list compartments that are selectable.  The literals compartment of an Enumeration is selectable, but none of the compartments in a Class in a class diagram can be selected.