Bug 113338 - [sort members] Usability improvements of sort members action
Summary: [sort members] Usability improvements of sort members action
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.2   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 3.2 M4   Edit
Assignee: Martin Aeschlimann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-21 05:52 EDT by Tobias Widmer CLA
Modified: 2005-12-13 09:52 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Widmer CLA 2005-10-21 05:52:31 EDT
I20051018-0932 + ZRH plugin export from 20051021

- The sort members action unnecessarily shows a dialog prompting whether to 
sort fields as well or not. This seems interrupting and more or less useless, 
as it resembles more a preference, which is set once and left untouched.
A better solution could be a MessageDialogWithToggle to attract the users 
attention to the potentially arising problems when sorting fields and guiding 
her to the members sort order preference page where she can configure the 
desired behavior

- Invoking sort members on a type with less than 2 members pops up a dialog 
telling that there were not enough members to sort. This is interrupting and 
unnecessary as well IMHO.
Comment 1 Dirk Baeumer CLA 2005-10-21 06:53:35 EDT
Benno, what is your take here ?
Comment 2 Benno Baumgartner CLA 2005-10-21 08:28:19 EDT
First: I wrote it exactly like that. Martin changed it. I think he thought that 
it does not belong to the preferences, but we have to ask him why he changed it.

Second: I will look into that. I agree that it is unnecessary, if a user wants 
to sort one or less members he should be aible to do so...
Comment 3 Martin Aeschlimann CLA 2005-10-23 00:10:58 EDT
The problem is that we don't have any other action preferences, and the sort
action just is (IMO) one of the least important actions. It would be just to
prominent and distracting on the preference page so that I think it's ok to take
the pain of an extra click on invocation.
Comment 4 John Arthorne CLA 2005-10-27 10:44:33 EDT
I also find the prompt to confirm annoying, since I sort members very
frequently.   If the only problem is where to put the preference, wouldn't it be
appropriate to appear on the "Members Sort Order" preference page?  Since the
confirmation dialog already has a link to that preferene page, it seems a
perfect fit.

I have never noticed the case of one member, but it seems doing nothing and not
showing any dialog is fine.  Since the member list is correctly sorted after the
action is executed (there was no failure), a dialog seems unnecessary. In any
case, I don't really care about this one because I would never think of sorting
a type with one member ;)
Comment 5 Tobias Widmer CLA 2005-10-27 10:52:09 EDT
Why not just add a single checkbox "sort fields" and a simple hint about its 
consequences on the members sort order preference page?

I do not see this issue as an action preference, but just an extension to the 
sort order
Comment 6 John Arthorne CLA 2005-10-27 11:04:23 EDT
I think the warning dialog makes sense as out of box experience, since users
might not be aware of the subtle errors that can be caused by sorting
initializers. There just needs to be a way to get rid of the prompt once you are
aware of the risk.  Adding a "remember my decision" checkbox to the dialog would
also work.
Comment 7 Markus Keller CLA 2005-11-01 10:35:16 EST
Martin, we have to discuss this again. The current solution only seems to
fulfill the goal of a 'nice' preference page, but OTOH heavily interferes with
common usage scenarios such as comment 4. 

The switch is IMO a preference and not an option, since the user wants to decide
this once and for all, and not on a case-by-case basis. Therefore, it belongs to
the preference page where all the other preferences affecting the sort order are
located.

I agree with the solution from comment 5. Another solution would be to just
never reorder the dangerous members and not to offer any preference (bug 109562
also didn't request one).

If the dialog box really has to stay, then the action would have to be renamed
to "Sort Members...".
Comment 8 Martin Aeschlimann CLA 2005-11-04 10:44:09 EST
Many actions show a dialog first. Why is this such a problem here?

I would like to add a 'don't ask me again' check box, but again we're missing a
preference to undo that. Maybe we shoulod think of introducing that.
Comment 9 Martin Aeschlimann CLA 2005-11-08 05:38:11 EST
added a 'do not show again' check box > 20051108
All 'do not show again' states can now be cleared on the Java preference page
Comment 10 Philip Mayer CLA 2005-12-13 06:20:41 EST
Verified implementation with a dialog including a "do not show again" checkbox, which can be cleared on the Java preference page.

Verified in I20051213-0010.