Bug 155230 - ViewmapProducer to deduce DefaultSizeAttributes if maximum size is set in gmfgraph for referenced figure
Summary: ViewmapProducer to deduce DefaultSizeAttributes if maximum size is set in gmf...
Status: RESOLVED FIXED
Alias: None
Product: GMF-Tooling (ARCHIVED)
Classification: Modeling
Component: Core (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Michael Golubev CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2006-08-25 10:36 EDT by Michael Golubev CLA
Modified: 2010-07-19 22:22 EDT (History)
0 users

See Also:


Attachments
Fixes the subj (6.08 KB, patch)
2006-08-25 12:38 EDT, Michael Golubev CLA
no flags Details | Diff
Fixes the last reopen note (75.47 KB, patch)
2006-09-01 10:28 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 Michael Golubev CLA 2006-08-25 10:36:00 EDT
Actually maximum size set in gmfgraph model for top-level nodes does not affect actual maximum size for figures at diagram -- instead user should adjust DefaultSizeAttributes for ViewMap in GenModel. 

Expected: It would be better if ViewmapProducer suggest the DefaultSizeAttributes in this case. 

It was originally requested in news group (see news:62b8f1200e62d9657aee959bbc1b8d77$1@www.eclipse.org).
Comment 1 Michael Golubev CLA 2006-08-25 10:37:38 EDT
NOTE: This may affect reconciler, because reconciler has general problems while processing derived values.
Comment 2 Michael Golubev CLA 2006-08-25 12:08:16 EDT
Hmm, it seems to be too greedy to always choose the biggest possible size :)
Thus, if preferred size is set, than producer should use _preferred_ size.
Comment 3 Michael Golubev CLA 2006-08-25 12:38:49 EDT
Created attachment 48746 [details]
Fixes the subj
Comment 4 Artem Tikhomirov CLA 2006-08-25 14:59:18 EDT
reviewed and committed.
Comment 5 Michael Golubev CLA 2006-08-30 10:57:17 EDT
It would be even better to have special purpose facet in gmfgraph to define the default size for some diagram node. 

The usage of Figure#preferredSize is not buggy itself, but it forces generated figure to have this given preferred size even if it contents does not fit it. 

Consider Actions in activity diagrams. There are 20+ types of actions that share most of visual properties. It is the reason for them to share the diagram node in the gmfgraph model. It is unconvenient to set DefaultSizeAttributes 20+ times for each action, so it still should be deduced somehow. 

But if it is configured via preferred size, than action figure lose ability to expand together with its name label.

Thus, it is proposed to defaine ad hoc facet class in the gmfgrph model and use its value to deduce default size.
Comment 6 Michael Golubev CLA 2006-09-01 10:28:21 EDT
Created attachment 49260 [details]
Fixes the last reopen note
Comment 7 Artem Tikhomirov CLA 2006-09-01 13:45:57 EDT
reviewed and committed
Comment 8 Eclipse Webmaster CLA 2010-07-19 22:22:58 EDT
[GMF Restructure] Bug 319140 : product GMF and component
Models was the original product and component for this bug