Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 306820 - [design] Glitch in CTabItem with rounded borders in IE
Summary: [design] Glitch in CTabItem with rounded borders in IE
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 M7   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-23 09:54 EDT by Tim Buschtoens CLA
Modified: 2010-03-31 13:21 EDT (History)
0 users

See Also:

Attachments
fix / workaround (2.18 KB, patch)
2010-03-30 11:48 EDT, Tim Buschtoens CLA 		ruediger.herrmann: iplog+
Details | Diff
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 Tim Buschtoens CLA 2010-03-23 09:54:18 EDT 
- Goto http://rap.eclipsesource.com/rapdemo/fancy?startup=controls
- Click any tab except the first two.
- Move the shell.
- The first tab-item appears to have two borders, one rounded, one not.
Comment 1 Tim Buschtoens CLA 2010-03-30 11:48:17 EDT 
Created attachment 163416 [details]
fix / workaround

What happens:
There is a bug in IE that clips VML-shapes incorrectly when using overflow:hidden. At least one row(top) and column(left) of the negative space are still visible. I found no reliable workaround. The ideal solution would be to change the implementations of setRoundRectLayout so they support shapes with only partially drawn strokes. (Should be possible using subpaths.)

Solution:
Since this would be a very expensive change, this fix works around the problem by changing AppearanceBase so that the combination of rounded borders and hidden edges does (almost) never occur. This works because the first item does not actually need a rounded border while not selected: The background is hardcoded to be transparent, and the visible rounded corner is part of the Folder.
Comment 2 Rüdiger Herrmann CLA 2010-03-31 13:21:05 EDT 
Applied patch to CVS HEAD