Community
Participate
Working Groups
Created attachment 104903 [details] Reproduced Problem When you set the background color on a TableItem, that color overrides the row selection color. In attach is a screen shoot with the problem visible. Can provide a sample code if needed.
Created attachment 105496 [details] Proposed patch for version 1.1 RC4.
Created attachment 105805 [details] Bugfix Altered version of the previous patch: - the actual problem was that _isItemSelected() always returned false as it was called with wrong parameters - fixed this - left out the passage below // Fix for bug when a colored cell propagates to all cell to end of table Don't exactly understand what this code is meant to solve. Could you explain some more?
Committed the third patch to CVS HEAD. I think this solves the issue described here. Hugo, can you confirm this thus I could close this bug?
Yes the problem seems to be solved in the CVS HEAD. Regarding the other problem, it's like this: Using a table color provider, if you set a color on the first cell, the cells on all the other columns will also be colored. I will attach a snippet reproducing this, but maybe filling a new bug is the most correct thing.
Created attachment 106216 [details] Reproduces the cell coloring problem,
(In reply to comment #4) > Yes the problem seems to be solved in the CVS HEAD. Regarding the other problem, > it's like this: Using a table color provider, if you set a color on the first > cell, the cells on all the other columns will also be colored. I will attach a > snippet reproducing this, > but maybe filling a new bug is the most correct thing. Yes, an new bug would help to keep the overview. I would like to ask you to 'transfer' the second issue to a new bug. Also I think the table widget itself is the error source. The snippet could be shortened and therefore would read easier if it would directly manipulate the tables' background color
Closing this since the initially raise issue is fixed. Please use distinct bugs to report distinct problems.
The problem seems to be occurring again in the latest CVS Head as well as in the RC1 build.
(In reply to comment #4) > [...] Using a table color provider, if you set a color on the first > cell, the cells on all the other columns will also be colored. I will attach a > snippet reproducing this, but maybe filling a new bug is the most correct thing. Opened a spearate bug: 251842: [Table] TableItem#setBackground( 0 ,Color) affects whole row https://bugs.eclipse.org/bugs/show_bug.cgi?id=251842
If the selection was set programmatically, everything worked OK. However changing the selection via user interaction, an eventually set background color was not overridden. Changes are in CVS HEAD
