Bug 135017 - Support resize policy for diagram nodes
Summary: Support resize policy for diagram nodes
Status: RESOLVED FIXED
Alias: None
Product: GMF-Tooling (ARCHIVED)
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.0   Edit
Assignee: Dmitry Stadnik CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks: 114061
  Show dependency tree
 
Reported: 2006-04-05 09:55 EDT by Dmitry Stadnik CLA
Modified: 2010-07-19 22:19 EDT (History)
1 user (show)

See Also:


Attachments
Fixes the subj (8.03 KB, patch)
2006-04-06 05:46 EDT, Michael Golubev CLA
no flags Details | Diff
Fixes the subj + tests + Direction.ALL as default direction constraint (27.82 KB, patch)
2006-04-06 17:41 EDT, Michael Golubev CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry Stadnik CLA 2006-04-05 09:55:39 EDT
When resize constraint is set to NONE for a node it should not be possible to 
resize it on diagram for example; see Port node in TaiPan.
Comment 1 Michael Golubev CLA 2006-04-06 02:23:43 EDT
This scr blocks #114061 that states (among other things): 

> Note, resize constraints are not yet taken from gmfgraph
> (gmfgraph.Node#resizeConstraints).

Comment 2 Michael Golubev CLA 2006-04-06 05:46:03 EDT
Created attachment 37857 [details]
Fixes the subj

Please note that taipan sample should be recreated (the both recreation of the taipan.gmfgen and regeneration of the editor code is required). 

Unfortunately, I can not produce this myself, because there is some other waiting patches for taipan which is not yet committed.
Comment 3 Michael Golubev CLA 2006-04-06 17:41:19 EDT
Created attachment 37944 [details]
Fixes the subj + tests + Direction.ALL as default direction constraint

This extends previous patch with tests and new functionality, which is implictly required. 
It allows nodes to be fully resizable by introducing new Direction.ALL into the gmfgraph model. The default value for gmfgraph.Node#direction is now set to ALL to confrom to the previous behavior.
Comment 4 Artem Tikhomirov CLA 2006-04-10 08:34:48 EDT
Refactored and committed. Patch includes "not_implemented_scr_129362_testViewmapRequiredPluginIDs" which doesn't compile, thus was thrown away.

The bugzilla remains open to resolve next issue: gmfgraph.Node#resizeConstraint was intended to hold *constraint*, or *limitation* of resize operation, not resize *allowance* as it's implemented in the proposed patch. IOW, resizeConstraint == NONE means 'resize whatever you like", and "VERTICAL" means 'resize only in horizontal directions'. We should either change DirectionMapper (as a side note, we may wish to change the name as well ;) and default value of resizeConstraints to NONE instead of ALL, or to rename graphdef.Node#resizeConstraint to smth like resizeDirections.
Comment 5 Artem Tikhomirov CLA 2006-04-14 10:33:00 EDT
Milestone updated.
Comment 6 Dmitry Stadnik CLA 2006-04-20 04:38:43 EDT
In draw2d these constants are used to provide allowance so inverting their 
meaning in our model will lead to confusion. And it's more convenient to think 
about what we add rather than what we subtract since it's may be not obvious 
what we subtract from )))
Comment 7 Dmitry Stadnik CLA 2006-04-20 04:59:43 EDT
-
Comment 8 Dmitry Stadnik CLA 2006-04-20 08:30:15 EDT
-
Comment 9 Richard Gronback CLA 2008-08-13 13:04:13 EDT
[target cleanup] 1.0 RC was the original target milestone for this bug
Comment 10 Eclipse Webmaster CLA 2010-07-19 22:19:22 EDT
[GMF Restructure] Bug 319140 : product GMF and component
Generation was the original product and component for this bug