Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 299629 - IE does not handle composites correctly
Summary: IE does not handle composites correctly
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.3   Edit
Hardware: All Windows All
: P3 major (vote)
Target Milestone: 1.3 M7   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-14 08:36 EST by Niels Lippke CLA
Modified: 2010-03-30 07:11 EDT (History)
1 user (show)

See Also:

Attachments
Java fragment describing the problem (1.13 KB, text/plain)
2010-01-14 08:36 EST, Niels Lippke CLA 		no flags Details
Proposed patch (859 bytes, patch)
2010-01-22 04:12 EST, Ivan Furnadjiev CLA 		no flags Details | Diff
fix fix (6.24 KB, patch)
2010-03-30 06:45 EDT, Tim Buschtoens CLA 		ruediger.herrmann: iplog+
Details | Diff
Show Obsolete (1) 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 Niels Lippke CLA 2010-01-14 08:36:29 EST 
Created attachment 156093 [details]
Java fragment describing the problem

The attached snippet shows 2 composites (A+B) which overlay each other. 
A has a text field and a label. B is just an empty composite for event 
receiving. If you click on the composites, the drawing order is reversed (so 
one time A is above B and the other time B above A). Both composites do not 
use a background.

While the snippet works as expected on Firefox (3.5) (namely, you cannot 
edit the text field while B is above A), is does not on IE (8), where you 
can always edit the text field.
Comment 1 Ivan Furnadjiev CLA 2010-01-20 10:32:42 EST 
It seems that if the upper composite is transparent ( no background color or background image set ) this composite does not handle mouse events in IE. There is a MS article related to a similar issue in IE 5 - http://support.microsoft.com/kb/262559. As a workaround, without testing myself, set a transparent image as background image on the composite B.
Comment 2 Ivan Furnadjiev CLA 2010-01-20 11:01:12 EST 
I've just tested the suggested workaround and it works fine. Need to think about proper generic solution of the problem.
Comment 3 Ivan Furnadjiev CLA 2010-01-22 04:12:14 EST 
Created attachment 156905 [details]
Proposed patch

Set the background-image property to "blank.gif" in Composite.default.css.
Comment 4 Ivan Furnadjiev CLA 2010-01-22 04:13:13 EST 
Applied patch to CVS HEAD.
Comment 5 Ivan Furnadjiev CLA 2010-03-30 06:42:11 EDT 
With the fix for bug 306695 there are some side effects.
Comment 6 Tim Buschtoens CLA 2010-03-30 06:45:49 EDT 
Created attachment 163393 [details]
fix fix

This time includes tests.
Comment 7 Rüdiger Herrmann CLA 2010-03-30 07:10:19 EDT 
Applied patch to CVS HEAD