Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 319903

Summary:	[Viewers] CheckboxTreeViewer - multiple issues
Product:	[Eclipse Project] Platform	Reporter:	Suganya Devi <suganyadevi.selvaraj>
Component:	UI	Assignee:	Platform UI Triaged <platform-ui-triaged>
Status:	RESOLVED INVALID	QA Contact:	Hitesh <hsoliwal>
Severity:	normal
Priority:	P3	CC:	bokowski, remy.suen, suganyadevi.selvaraj
Version:	3.5.2
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Suganya Devi

2010-07-14 15:06:37 EDT

Build Identifier: 3.5.2

1. Check/Uncheck does not seem to visually select (highlight) the row. Say row 3 is currently selected and check the checkbox in row 1.  Though it checks row 1, it does not visually highlight row 1 (row 3 is still highlighted).

2. CheckboxTreeViewer.setChecked() is not firing CheckStateChangedEvent

Reproducible: Always

Comment 1 Suganya Devi

2010-07-14 15:08:55 EDT

Additional Info:
I have a CheckboxTreeViewer with multiple columns (Check+table+tree viewer)

Comment 2 Remy Suen

2010-07-14 22:03:22 EDT

(In reply to comment #0)
> 1. Check/Uncheck does not seem to visually select (highlight) the row. Say row
> 3 is currently selected and check the checkbox in row 1.  Though it checks row
> 1, it does not visually highlight row 1 (row 3 is still highlighted).

I think this is the behaviour of the platform. Do you get the same result if you use regular SWT code without JFace?

> 2. CheckboxTreeViewer.setChecked() is not firing CheckStateChangedEvent

I'm pretty sure this is working as intended.

Comment 3 Hitesh

2010-07-15 08:10:29 EDT

(In reply to comment #1)
> Additional Info:
> I have a CheckboxTreeViewer with multiple columns (Check+table+tree viewer)

Remy is right about the two issues you mention.

>1) 
I assume by 'highlight' you refer to the hover-highlighting. This can be seen in MS Windows 7 , but I doubt it is there in Win XP. Anyways the viewer behaves much like the corresponding SWT widget.

>2)
It works as expected. It is highly unlikely this behaviour will change.

Comment 4 Hitesh

2010-07-19 04:05:44 EDT

Marking as not a bug.