Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 363279

Summary: [SysML Block Definition Diagram] Element icon of flowport in flowports compartment does not reflect flowport direction
Product: [Modeling] Papyrus Reporter: Alain Le Guennec <alain.leguennec>
Component: CoreAssignee: Remi Schnekenburger <rschnekenburger>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: eclipse-bugzilla, yann.tanguy
Version: 0.8.1   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Screenshot illustrating icon inconsistency
none
mylyn/context/zip none

Description Alain Le Guennec CLA 2011-11-09 05:24:57 EST
Created attachment 206658 [details]
Screenshot illustrating icon inconsistency

The element icon of flowport in flowports compartment does not reflect flowport direction. It systematically is the icon of a bi-directional flowport.
Similar issue for icon of flowproperty in flowproperties compartment of a flowspecification.
See attached snapshot, where icons are correct in model explorer, but not quite in compartments on the BDD.
There apparently are different inconsistent ways of retrieving the icon for such SysML elements.
Comment 1 Alain Le Guennec CLA 2011-11-09 06:03:43 EST
Might be related to bug #352555 ?
Comment 2 Remi Schnekenburger CLA 2011-11-30 07:57:06 EST
I updated the way icons are found. I used a common label provider for elements, which uses the same customizations as in Model Explorer. Thus, icons should be coherent between model explorer and diagrams.

fixed in r6329
Comment 3 Remi Schnekenburger CLA 2011-11-30 07:57:09 EST
Created attachment 207717 [details]
mylyn/context/zip
Comment 4 Yann Tanguy CLA 2011-12-14 04:55:01 EST
(In reply to comment #2)
> I updated the way icons are found. I used a common label provider for elements,
> which uses the same customizations as in Model Explorer. Thus, icons should be
> coherent between model explorer and diagrams.
> 
> fixed in r6329

Merged in trunk in r6458.