Bug 209802 - [ImageSupport] Missing label icons in the editor area
Summary: [ImageSupport] Missing label icons in the editor area
Status: RESOLVED FIXED
Alias: None
Product: GMF-Runtime
Classification: Modeling
Component: General (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows XP
: P1 major
Target Milestone: 2.1   Edit
Assignee: Cherie Revells CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 214160
Blocks: 189608 211851
  Show dependency tree
 
Reported: 2007-11-14 09:31 EST by Jacques LESCOT CLA
Modified: 2008-08-13 13:08 EDT (History)
3 users (show)

See Also:


Attachments
Screenshot that describe the bug (20.95 KB, image/png)
2007-11-14 09:31 EST, Jacques LESCOT CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jacques LESCOT CLA 2007-11-14 09:31:42 EST
Created attachment 82874 [details]
Screenshot that describe the bug

Build ID: I20071101-2000

Steps To Reproduce:
Since the recent changes (WrapLabel has been replaced by WrappingLabel) it seems that icon label are no longer visible in the editor area, whereas it is visible in the outline thumbnail view. The problem is present in the ecore diagram provided as example and we have similar issue in our GMF editors.

More information:
Comment 1 Anthony Hunter CLA 2007-11-14 09:51:14 EST
The ECore editor is generated by the tooling. Cherie, was there any changes the tooling needed to adopt?
Comment 2 Cherie Revells CLA 2007-11-14 09:59:06 EST
Yes, the tooling should make changes to generate with the new WrappingLabel otherwise they could have some issues.  I am hoping there won't be too many issues with using the deprecated WrapLabel, but it's impossible to guarantee that since it extends the new WrappingLabel.  I'll take a look at the ECore editor to see what the problem is.
Comment 3 Jacques LESCOT CLA 2007-11-14 10:20:28 EST
In fact, I initially identified this problem in the Ecore Tools component I am working on. In my case, I manually replaced all the WrapLabel occurences with WrappingLabel references : I did it manually because I could not regenerate as the templates were not updated in consequence to support WrappingLabel. So the problem is also present with icon labels using the WrappingLabel class. I just tried the ecore editor example provided with GMF to see whether the problem was only on my side, which was not the case.
While trying to regenerate using *.gmfgen model, it seems that templates still generate dependencies to the WrapLabel class : is it planned to completely remove those references into WrappingLabel ones ?
Comment 4 Cherie Revells CLA 2007-11-14 10:44:50 EST
There is an enhancement logged against the generation team to make these changes.  See Bug 206582 .
Comment 5 Cherie Revells CLA 2007-11-14 11:09:04 EST
I just tried the GMF ECore Editor using GMF from HEAD and the icon appears fine for me.  I have never seen this problem before.  Are you sure there is not an IconProvider interfering somehow?
Comment 6 Cherie Revells CLA 2007-11-14 11:18:28 EST
Jacques, can you try putting a breakpoint in EPackageNameEditPart.getLabelIcon() and then try creating an EPackage on the diagram?
Comment 7 Jacques LESCOT CLA 2007-11-14 11:33:54 EST
I can reproduce the bug while installing a new Eclipse 3.4M3 version with necessary dependencies (GMF 2.1M3, EMF 2.4M3, GEF 3.4M3, MDT OCL 1.2M3, EMF Query/Transaction/Validation 1.2M3).

When I add a breakpoint on the EPackageNameEditPart.getLabelIcon() method, it correctly stops at the line :
- the parserElement is the model element associated with the EPackage I have created
- it correcly returns an Image (you can see in the attached screenshot that there is a space between the label and the parent container corresponding to the icon size, but it is not visible.
Comment 8 Jacques LESCOT CLA 2007-11-14 11:36:52 EST
I found the origin of the bug : antialiasing.
When I disable antialiasing, icons are correctly displayed. I am running under Windows XP, and I had never had problems with antialiasing before.
Comment 9 Cherie Revells CLA 2007-11-14 11:48:42 EST
That's odd.  I have Windows XP as well and it works fine with antialiasing for me.
Comment 10 Anthony Hunter CLA 2007-11-14 17:15:00 EST
(In reply to comment #9)
> That's odd.  I have Windows XP as well and it works fine with antialiasing for
> me.
> 

I have duplicated this issue on my XP machine.
Comment 11 Cherie Revells CLA 2007-11-15 09:56:08 EST
Ok, I can reproduce this now too.  It has something to do with one of the GMF dependencies.  With the current GMF code and a 3-month old Eclipse target, the problem does not exist for me, but with the lastest Eclipse target it does.  I will investigate.  It doesn't seem to have to do with the new WrappingLabel though, I can reproduce it with the old WrapLabel as well.
Comment 12 Cherie Revells CLA 2008-01-02 16:40:39 EST
Reproduced in SWT snippet.  Created Bug 214160 against SWT component.
Comment 13 Anthony Hunter CLA 2008-03-28 14:15:44 EDT
Marking resolved since the SWT bug has been fixed.
Comment 14 Richard Gronback CLA 2008-08-13 13:08:53 EDT
[target cleanup] 2.1 M5 was the original target milestone for this bug
Comment 15 Eclipse Webmaster CLA 2010-07-19 12:30:27 EDT
[GMF Restructure] Bug 319140 : product GMF and component Runtime Diagram was the original product and component for this bug