Bug 131609 - [validation] Problems selecting enable/disable on validation preference page
Summary: [validation] Problems selecting enable/disable on validation preference page
Status: CLOSED FIXED
Alias: None
Product: WTP Common Tools
Classification: WebTools
Component: wst.validation (show other bugs)
Version: 1.5   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 1.5 RC2   Edit
Assignee: Valentin Baciu CLA
QA Contact:
URL:
Whiteboard:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2006-03-13 17:04 EST by Lawrence Mandel CLA
Modified: 2006-06-26 13:37 EDT (History)
1 user (show)

See Also:


Attachments
Proposed patch (19.60 KB, patch)
2006-03-16 16:43 EST, Valentin Baciu CLA
no flags Details | Diff
New disabled icon (896 bytes, image/gif)
2006-03-16 16:44 EST, Valentin Baciu CLA
no flags Details
New enabled icon (343 bytes, image/gif)
2006-03-16 16:45 EST, Valentin Baciu CLA
no flags Details
Tasks screenshot (78.72 KB, image/bmp)
2006-03-29 11:41 EST, Valentin Baciu CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lawrence Mandel CLA 2006-03-13 17:04:19 EST
The validation preference page was updated in the 1.5 stream to allow individual validators to be disabled for the build or the manual validation option. When on the preference page I try to select to disable the XML validator for both build and manual validation. No options are presented. I can only change an option once I select the validator name "XML Validator". This is unintuitive for users. It appears as though the options cannot be changed.

A user should be able to change any option by simply clicking on it. There does not seem to be any need to require them to click on the validator name before changing an option.
Comment 1 Lawrence Mandel CLA 2006-03-13 17:08:55 EST
A second improvement (suggested by Valentin) is that clicking on the enabled/disabled image should toggle the switch.
Comment 2 Valentin Baciu CLA 2006-03-16 16:43:54 EST
Created attachment 36446 [details]
Proposed patch

Here's a patch which uses simulated check-boxes instead of the combo cell editors. The patch is not 100% done but I think it moves things in the right direction. 

To be completed:
- improved icons if desired
- provide different icons when a row is selected
- provide different icons when the list is disabled

I hope this helps. Regards, Valentin
Comment 3 Valentin Baciu CLA 2006-03-16 16:44:38 EST
Created attachment 36447 [details]
New disabled icon
Comment 4 Valentin Baciu CLA 2006-03-16 16:45:05 EST
Created attachment 36448 [details]
New enabled icon
Comment 5 Lawrence Mandel CLA 2006-03-16 17:36:43 EST
Excellent improvement Valentin! The preference page is much more usable with this patch.

I do agree that something needs to be done when the row is selected. The row currently turns blue and makes the checkboxes look greyed out and, for lack of a better word, crappy.
Comment 6 Vijay Bhadriraju CLA 2006-03-17 11:59:37 EST
Valentin as you are fixing this defect it makes more sense to assign it to you. I will be on the CC and send me any new patches and i will review and commit them. Thanks
Comment 7 Valentin Baciu CLA 2006-03-27 18:23:16 EST
Opened https://bugs.eclipse.org/bugs/show_bug.cgi?id=133441 against SWT as it seems that the blury images when a row is selected may be a Table rendering issue. Trees seem to work OK.
Comment 8 Valentin Baciu CLA 2006-03-29 11:40:18 EST
I'm just going through the things that were to be done:
- improved icons and disabled icons were provided as part of the fix for https://bugs.eclipse.org/bugs/show_bug.cgi?id=132228
- the explanation for the blurry icons when a row is selected is provided here https://bugs.eclipse.org/bugs/show_bug.cgi?id=133441. It appears that it is a platform behaviour to draw a shade around images in a table item.

I wonder if we can consider this done and mark it as fixed? Or do we want to explore using a tree? The marker views (problems, tasks, etc) use a tree viewer and images seems to be painted nicely. See the attached screen shot.
Comment 9 Valentin Baciu CLA 2006-03-29 11:41:05 EST
Created attachment 37215 [details]
Tasks screenshot
Comment 10 John Lanuti CLA 2006-04-24 16:32:18 EDT
This patch is released for WTP 1.5 RC1.
Comment 11 Lawrence Mandel CLA 2006-06-26 13:37:25 EDT
Verified on WTP 1.5 RC6.
Comment 12 Lawrence Mandel CLA 2006-06-26 13:37:33 EDT
Closing bug.