Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 352549 - [General] LineWidth of a box symbol only applies to compartment separator lines, not to the box boundary
Summary: [General] LineWidth of a box symbol only applies to compartment separator lin...
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Diagram (show other bugs)
Version: 0.8.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 353700 353701
  Show dependency tree
 
Reported: 2011-07-20 05:08 EDT by Alain Le Guennec CLA
Modified: 2015-04-24 09:04 EDT (History)
2 users (show)

See Also:

Attachments
Screenshot for LineWidth issue. (1.11 MB, image/bmp)
2011-07-20 05:08 EDT, Alain Le Guennec CLA 		no flags Details
View All 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 Alain Le Guennec CLA 2011-07-20 05:08:15 EDT 
Created attachment 199964 [details]
Screenshot for LineWidth issue.

The LineWidth appearance property of a box symbol only applies to compartment separator lines, not to the box boundary.
Look at attached screenshot, in which all box symbols (blocks, parts) have been given a LineWidth of 10.
Comment 1 Yann Tanguy CLA 2011-08-03 04:19:43 EDT 
In r5175 : 
- BorderNodeEditPart and NodeEditPart have been modified to support lineWidth modification
- BorderNodeEditPart and NodeEditPart lineStyle support have been modified (improvement in refresh)
- IPapyrusNodeFigure and PapyrusNodeFigure now support lineWidth modification

Note that some complex figure (ex: Package) or figure that do not inherit from PapyrusNodeFigure have to be corrected.
Note that the EditPart generated code mask setLineWidth() (and setLineType()) implementation. The code generator should be modified (either replace the implementation by a call to super class method or totally remove it) and diagrams regenerated, or diagrams manually modified.
Comment 2 Yann Tanguy CLA 2011-08-03 07:45:17 EDT 
(In reply to comment #1)
> In r5175 :
> - BorderNodeEditPart and NodeEditPart have been modified to support lineWidth
> modification
> - BorderNodeEditPart and NodeEditPart lineStyle support have been modified
> (improvement in refresh)
> - IPapyrusNodeFigure and PapyrusNodeFigure now support lineWidth modification

Merge in trunk : r5181.
Comment 3 Alain Le Guennec CLA 2011-08-18 12:19:40 EDT 
I wanted to check the fix, but unfortunately, the "Appearance" property tab does no more allow to set the line width of a symbol (only its line color)... Is that on purpose?
Comment 4 Yann Tanguy CLA 2011-08-31 10:50:43 EDT 
(In reply to comment #3)
> I wanted to check the fix, but unfortunately, the "Appearance" property tab does
> no more allow to set the line width of a symbol (only its line color)... Is that
> on purpose?

Yes, this is disabled in Papyrus until all diagrams support the feature.
Comment 5 Arnaud Cuccuru CLA 2013-03-13 11:52:27 EDT 
Not able to reproduce the bug under Kepler M5
Comment 6 Arnaud Cuccuru CLA 2013-03-13 12:41:39 EDT 
Reopening the bug. This works for SysML diagrams, but not for UML diagrams. In UML diagrams, the LineWidth appearance property of a box symbol only applies to compartment separator lines.
Comment 7 Camille Letavernier CLA 2015-04-24 09:04:35 EDT 
This works as expected in 1.1

I close the task