Bug 441801 - Optimize the way element pinning is encoded
Summary: Optimize the way element pinning is encoded
Status: NEW
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 1.0.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2014-08-14 10:11 EDT by Pierre-Charles David CLA
Modified: 2014-08-14 11:49 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pierre-Charles David CLA 2014-08-14 10:11:31 EDT
When element pinning was introduced, we planned for more sophisticated versions of the feature and decided to encode the "pinned state" like this instead of using a simple boolean flag:

<arrangeConstraints>KEEP_LOCATION</arrangeConstraints>
<arrangeConstraints>KEEP_SIZE</arrangeConstraints>
<arrangeConstraints>KEEP_RATIO</arrangeConstraints>

This is 3 lines and almost 200 bytes for a simple flag, repeated for each pinned diagram element. This makes aird models bigger, and (although I have not tested) slower to load an serialize. We should replace this with a simple boolean flag and do the appropriate migration.
Comment 1 Esteban DUGUEPEROUX CLA 2014-08-14 11:49:28 EDT
Another concept which takes many spaces in aird resource is Style Customisation feature as it store computed StyleDescription in aird resource.