Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 301709 - IE can clip 2 pixels too much
Summary: IE can clip 2 pixels too much
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.3   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.3 M6   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-03 11:02 EST by Tim Buschtoens CLA
Modified: 2010-02-09 06:17 EST (History)
0 users

See Also:


Attachments
Screenshots of the issues (30.63 KB, image/png)
2010-02-03 11:02 EST, Tim Buschtoens CLA
no flags Details
fix (5.31 KB, text/plain)
2010-02-08 12:42 EST, Tim Buschtoens CLA
ruediger.herrmann: iplog+
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Buschtoens CLA 2010-02-03 11:02:48 EST
Created attachment 158065 [details]
Screenshots of the issues

This happens in IE whenever "prepareEnhancedBorder" (in Widget.js or GraphicsMixin.js respectively) was called, but no "enhanced border" (i.e. Inset/Outset or a rounded Border) is actually used. This is always the case if a widget has a gradient, but no (or only a simple) border.

To reproduce in controls demo:
- Goto tab "Composite"
- Activate gradient
- Resize the shell horizontally so the widgets don't fit into the composite.

See attachment for the result. Left is FF (corrent), IE on the right.

This is a qooxdoo bug that has not really anything to do with the GraphicsMixin other than it is triggered by it. The problem lies within Widget.js in "_renderRuntimeWidth" (only used in IE). To solve it, we could either use the same functions in IE as the other browser (but have a potential loss in performance), or set a flag to indicate wether these 2px substraction is needed.
Comment 1 Tim Buschtoens CLA 2010-02-08 12:42:32 EST
Created attachment 158492 [details]
fix

Needs to rebuild qx.js. Test updated.
Comment 2 Rüdiger Herrmann CLA 2010-02-09 06:17:05 EST
Applied patch to CVS HEAD