Bug 364821 - Grid lines problem in IE for SWT table used in RAP
Summary: Grid lines problem in IE for SWT table used in RAP
Status: RESOLVED WORKSFORME
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: PC Unix All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: needinfo
Depends on:
Blocks:
 
Reported: 2011-11-25 06:38 EST by Amandeep Chahal CLA
Modified: 2011-12-05 05:22 EST (History)
2 users (show)

See Also:

Attachments
Screenshot of table view in IE (130.33 KB, image/jpeg)
2011-11-25 06:39 EST, Amandeep Chahal CLA 		no flags Details
TableViewer demo in IE (153.95 KB, image/jpeg)
2011-11-28 03:08 EST, Amandeep Chahal CLA 		no flags Details
TableViewer demo in Mozilla (115.62 KB, image/jpeg)
2011-11-28 03:09 EST, Amandeep Chahal CLA 		no flags Details
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 Amandeep Chahal CLA 2011-11-25 06:38:35 EST 
Build Identifier: 3.7

For SWT table used in RAP, in IE the horizontal grid lines are black and the vertical are not. Expected behavior is both should be non - black as the color as a non black color has been set. Also the horizontal grid lines for non existent rows also appear. Attached screen shot for reference.

Reproducible: Always

Steps to Reproduce:
1. Add SWT table to view. Set a background color for the table.
2. Set gridLinesVisible to true.
3. Grid lines show up incorrectly.
Comment 1 Amandeep Chahal CLA 2011-11-25 06:39:17 EST 
Created attachment 207526 [details]
Screenshot of table view in IE
Comment 2 Tim Buschtoens CLA 2011-11-25 07:42:58 EST 
Works for me. How did you set the color? The BackgroundColor has no effect on the gridlines, you need to use the theming for those.
Comment 3 Amandeep Chahal CLA 2011-11-28 03:06:16 EST 
(In reply to comment #2)
> Works for me. How did you set the color? The BackgroundColor has no effect on
> the gridlines, you need to use the theming for those.

Agreed. If you set it using the theme, the vertical and horizontal lines differ. This can also be observed in the RAP demos' TableViewer example : http://rap.eclipsesource.com/rapdemo/.

Attaching screenshot of the TableViewer in IE and Mozilla.
Comment 4 Amandeep Chahal CLA 2011-11-28 03:08:31 EST 
Created attachment 207589 [details]
TableViewer demo in IE
Comment 5 Amandeep Chahal CLA 2011-11-28 03:09:00 EST 
Created attachment 207590 [details]
TableViewer demo in Mozilla
Comment 6 Tim Buschtoens CLA 2011-11-28 06:16:26 EST 
Sure, horizontal/vertical lines differ if they are themed differently. In the default theme it looks like this:

Table-GridLine {
  color: #dedede;
}

Table-GridLine:horizontal {
  color: transparent;
}

Therefore vertical lines are visible, horizontal lines transparent.

Or did you mean they differ between the browser? The IE screenshop looks broken (black horizontal lines), but it seems to be IE6, right? (IE6 is no longer supported in 1.5)
Comment 7 Amandeep Chahal CLA 2011-11-28 07:17:18 EST 
Thats right. They differ between the 2 browsers. I'm using IE version 6.
Comment 8 Ralf Sternberg CLA 2011-11-29 08:59:07 EST 
(In reply to comment #7)
> Thats right. They differ between the 2 browsers. I'm using IE version 6.

As Tim pointed out, IE 6 is not supported in RAP 1.5 anymore (bug 345122). Can you also reproduce this problem with a newer IE version?
Comment 9 Amandeep Chahal CLA 2011-12-05 04:48:55 EST 
(In reply to comment #8)
> (In reply to comment #7)
> > Thats right. They differ between the 2 browsers. I'm using IE version 6.
> 
> As Tim pointed out, IE 6 is not supported in RAP 1.5 anymore (bug 345122). Can
> you also reproduce this problem with a newer IE version?

I don't have a newer version of IE now. Nor can I install it. Will get back whenever we get upgraded to a newer version. Thanks.
Comment 10 Ralf Sternberg CLA 2011-12-05 05:22:09 EST 
Closing this bug because it is likely a IE6-only issue. Please reopen when you can reproduce this problem on IE7+.