Bug 172574 - [IDE] DeleteProjectDialog inconsequent selection behavior
Summary: [IDE] DeleteProjectDialog inconsequent selection behavior
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: All Linux-GTK
: P3 trivial (vote)
Target Milestone: 3.3 M7   Edit
Assignee: Susan McCourt CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-02-01 16:29 EST by Benjamin Muskalla CLA
Modified: 2007-05-01 12:01 EDT (History)
0 users

See Also:


Attachments
deletenoundoclick.diff (1.52 KB, patch)
2007-02-01 16:30 EST, Benjamin Muskalla CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Muskalla CLA 2007-02-01 16:29:58 EST
Normally, choosing an option in an radio group can be done by clicking on the labels instead clicking on the buttons itself. As the "(Contents cannot be restored using "Undo")" text is in an extra label, clicking on it does nothing. I see me often clicking on that label to select the corresponding option.

Attached is a little patch to select the correct radio when clicking on the extra label.
Comment 1 Benjamin Muskalla CLA 2007-02-01 16:30:38 EST
Created attachment 58064 [details]
deletenoundoclick.diff

Here it is :-)
Comment 2 Susan McCourt CLA 2007-04-24 13:20:12 EDT
fix for M7
Comment 3 Susan McCourt CLA 2007-04-24 16:28:54 EDT
released in HEAD >20070424.
I modified the patch slightly (I prefer a MouseAdapter to keep the code to a minimum).

Note that you don't get the same visual cue that selecting the label will update the radio state that you get if you move the mouse over the radio text (on Windows).  Also there is dead space in the indent and in a few pixels between the two lines.  So it's a bit odd, but I think helpful enough for most users to incorporate the fix.  If we get complaints about this behavior (lack of visual emphasis, etc.) I will yank it.
Comment 4 Susan McCourt CLA 2007-04-24 16:29:36 EDT
forgot to say...thanks for the patch  ;-)
Comment 5 Susan McCourt CLA 2007-05-01 12:01:20 EDT
verified on Win XP, I20070430-0800