Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 176993 - [Wizards/BiDi] JFace Wizard background image not mirrored for Bidi Languages
Summary: [Wizards/BiDi] JFace Wizard background image not mirrored for Bidi Languages
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P1 major (vote)
Target Milestone: 3.3 M7   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-03-12 10:41 EDT by Michael Burkhart CLA
Modified: 2007-05-01 16:21 EDT (History)
5 users (show)

See Also:


Attachments
Bidi Wizard picture (60.60 KB, image/pjpeg)
2007-03-12 10:42 EDT, Michael Burkhart CLA
no flags Details
jface jar with flipped images (27.06 KB, application/x-zip-compressed)
2007-03-15 19:31 EDT, Kit Lo CLA
no flags Details
patch (36.48 KB, patch)
2007-04-03 10:47 EDT, Tod Creasey CLA
no flags Details | Diff
JFace Bidi example updated with new paths (20.39 KB, application/octet-stream)
2007-04-03 10:48 EDT, Tod Creasey CLA
no flags Details
A wizard with no images for testing (6.12 KB, application/octet-stream)
2007-04-30 12:51 EDT, Tod Creasey CLA
no flags Details
Better test example (5.16 KB, application/zip)
2007-05-01 16:21 EDT, Tod Creasey CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Burkhart CLA 2007-03-12 10:41:09 EDT
When executing Bidi testing for some of our product components we noticed that when using the org.eclipse.jface.wizard.WizardPage class the background image in the top panel of the wizard dialog is not mirrored. When running in non-Bidi languages the image appears to fade towards the right and then stops at the edge of the pane, but in a bidi mode where it is shifted to the left side of the pane, it looks bad that is starts from the left and then just stops when it reaches the top of the pane. Please see attached image of bidi and non-bidi images.

This can be recreated by creating a standard plugin project, selecting a custom plugin wizard, then click next and then finish. With this new project in your workspace, create a new run configuration and set a new Program Argument of "-nl ar" to launch with in Arabic. Once the new SDK instance is launched right click in the workspace and select new, then other, then Sample Wizards then Multi page editor file. The wizard that appears will show the same problem in bidi.

This is considered a serious bidi problem for our product, settnig to Major.
Comment 1 Michael Burkhart CLA 2007-03-12 10:42:24 EDT
Created attachment 60554 [details]
Bidi Wizard picture
Comment 2 Tod Creasey CLA 2007-03-13 20:55:32 EDT
Alternative icons need to be supplied by the wizard suppliers in thier langiage packs.

Is this an issue in any SDK wizards with the BIDI language packs installed?
Comment 3 Kit Lo CLA 2007-03-15 19:31:17 EDT
Created attachment 61027 [details]
jface jar with flipped images

Tod, here is the jface jar with the flipped images. However, I still couldn't get the jface wizard to display the flipped banner image. Please check if jface has the $nl$ code to search the flipped images.
Comment 4 Tod Creasey CLA 2007-03-19 08:27:43 EDT
Reopening to investigate
Comment 5 Tod Creasey CLA 2007-03-21 14:57:52 EDT
JFaceResources initializeDefaultImages is not using the nl prefix.
Comment 6 Tod Creasey CLA 2007-03-21 16:06:30 EDT
This turns out to be more complicated that just adding a $nl$ prefix - we actually need to migrate our lookup to use the OSGI as we did in the workbench.

We will address is in M7.
Comment 7 Philipe Mulet CLA 2007-03-21 16:35:55 EDT
Please make sure to attach the 3.3 patch, and risk assesment for a 3.2.2
backport.
Comment 8 Tod Creasey CLA 2007-04-03 10:45:14 EDT
Fixed in build >200700403. We were not using the bundle based lookup at all in JFace so I moved TitleAreaDialog, Dialog and PreferenceDialog over to an OSGi compliant lookup.

This required me to move the icons so I have written the code such that if the new icons are not found the old ones can be looked up.

We were also not using the .png version of the preference dialog icon which has a transparent background by default - I have updated to use it.

I will attach an updated version of Kits example that reflects the new paths (thanks very much Kit).

I would typify this as a low risk change should someone want to patch 3.2.2. I will attach a patch for 3.3 but this will be out of date for 3.2.2 as there have been some intervening changes.
Comment 9 Tod Creasey CLA 2007-04-03 10:47:14 EDT
Created attachment 62788 [details]
patch
Comment 10 Tod Creasey CLA 2007-04-03 10:48:56 EDT
Created attachment 62789 [details]
JFace Bidi example updated with new paths
Comment 11 Tod Creasey CLA 2007-04-30 12:51:34 EDT
Created attachment 65398 [details]
A wizard with no images for testing
Comment 12 Tod Creasey CLA 2007-05-01 16:21:12 EDT
Created attachment 65529 [details]
Better test example
Comment 13 Tod Creasey CLA 2007-05-01 16:21:56 EDT
Verified in I20070501-0010