490444 – The Copy Appearance Action should consider the last selected element as base style

Bug 490444 - The Copy Appearance Action should consider the last selected element as base style

Summary: The Copy Appearance Action should consider the last selected element as base ...

Status:	CLOSED FIXED

Alias:	None

Product:	Sirius
Classification:	Modeling
Component:	Diagram (show other bugs)
Version:	3.1.0
Hardware:	All All

Importance:	P3 enhancement (vote)
Target Milestone:	4.0.0M7
Assignee:	Florian Barbin
QA Contact:	Laurent Redor

URL:
Whiteboard:
Keywords:	triaged

Depends on:
Blocks:

Reported:	2016-03-25 11:05 EDT by Florian Barbin
Modified:	2016-06-24 08:03 EDT (History)
CC List:	2 users (show)

See Also:	Gerrit Change Gerrit Change Gerrit Change Gerrit Change Git Commit Git Commit Git Commit Gerrit Change Git Commit Gerrit Change Git Commit

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Barbin

2016-03-25 11:05:09 EDT

Currently, the Copy Appearance Properties Action is not consistent with the Size Both Action which uses the last selected element as base size. The Copy Appearance Properties Action uses the first selected element as base style. Furthermore, the last selected element is highlighted in a way that suggests this one is the reference to copy the style or the size.

Comment 1 Eclipse Genie

2016-04-08 11:54:27 EDT

New Gerrit change created: https://git.eclipse.org/r/70270

Comment 2 Eclipse Genie

2016-04-25 12:10:31 EDT

Gerrit change https://git.eclipse.org/r/69331 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=b3953194b7606d240ece76eca8a31e18f0e112e3

Comment 3 Eclipse Genie

2016-04-25 12:17:35 EDT

Gerrit change https://git.eclipse.org/r/69577 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=9f050e77882080dab460821481b549c1117c9d10

Comment 4 Eclipse Genie

2016-04-25 12:17:46 EDT

Gerrit change https://git.eclipse.org/r/69576 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=c6c91e00ce7a10530c11300475d5aa60ce284008

Comment 5 Eclipse Genie

2016-04-29 03:57:26 EDT

New Gerrit change created: https://git.eclipse.org/r/71665

Comment 6 Eclipse Genie

2016-04-29 03:58:28 EDT

Gerrit change https://git.eclipse.org/r/71665 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=b2803c39513bbb18d557e913b6e646f7befb19ef

Comment 7 Florian Barbin

2016-05-26 07:33:29 EDT

To validate this issue:
Create a simple diagram with several graphical elements
Customize the style of one element
Select several element by selection the customize one at the end
Apply the copy appearance action
All selected elements should have the same style than the customized one.

Comment 8 Eclipse Genie

2016-05-26 11:32:44 EDT

New Gerrit change created: https://git.eclipse.org/r/73734

Comment 9 Laurent Redor

2016-05-26 11:37:21 EDT

Verified on Sirius 4.0.0 RC1 (4.0.0.201605180923)

Except a small modification in documentation (gerrit https://git.eclipse.org/r/73734 )

Comment 10 Eclipse Genie

2016-05-27 08:56:11 EDT

Gerrit change https://git.eclipse.org/r/73734 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=56c4346d342b5a5673ead39428ba4547fab55b0a

Comment 11 Pierre-Charles David

2016-06-24 08:03:25 EDT

Available in Sirius 4.0.0.