Bug 176419 - [Progress] Unlabelled progress monitors shiuld have a default value in the blocked jobs dialog
Summary: [Progress] Unlabelled progress monitors shiuld have a default value in the bl...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M7   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard:
Keywords: polish
: 181165 (view as bug list)
Depends on: 182568
Blocks:
  Show dependency tree
 
Reported: 2007-03-05 16:10 EST by Martin Aeschlimann CLA
Modified: 2007-04-30 12:40 EDT (History)
1 user (show)

See Also:


Attachments
screenshot (12.80 KB, image/png)
2007-03-05 16:11 EST, Martin Aeschlimann CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Aeschlimann CLA 2007-03-05 16:10:55 EST
20070503

The following blocking dialog was reached on 'Save' while a build was running.

See screenshot contains the following UI issues:

- No label for first entry. What about 'User operation' 
- The description should use single quotes for consistency with other labels in Eclipse.
Comment 1 Martin Aeschlimann CLA 2007-03-05 16:11:38 EST
Created attachment 60284 [details]
screenshot
Comment 2 Martin Aeschlimann CLA 2007-03-05 16:17:34 EST
- the dialog title should better not form a sentence with a verb, but more something like 'Blocked User Operation'
Comment 3 Tod Creasey CLA 2007-03-08 10:09:56 EST
We are taking the current task name from the monitor but it is becoming clearer that many of these cases are happening before the monitor is initialized. Some default text is a good idea.

Thanks for the feedback Martin.
Comment 4 Martin Aeschlimann CLA 2007-04-10 13:19:52 EDT
*** Bug 181165 has been marked as a duplicate of this bug. ***
Comment 5 Philipe Mulet CLA 2007-04-11 07:30:30 EDT
Or 'Pending/Waiting User Operation' ? Blocked feels a bit too harsh.

Shouldn't we also retaliate against progress monitors with missing labels ?
Comment 6 Tod Creasey CLA 2007-04-11 07:48:00 EDT
We should certainly log seperate bugs to components whose monitors block without any task information for us to show.

Comment 7 Tod Creasey CLA 2007-04-16 11:51:30 EDT
See Bug 182568 for the offending empty progress monitor.

I have changed our logic to check for null as well as empty Strings. Currently (i.e. since 3.0) we take the shell title for the name of the unlabeled entry field. If you think this is not the right direction please log a bug for your suggested change and we can make it take Philippes suggested wording.

Fixed for build >20070316

To replicate do the following
1) Import a lot of plug-ins and fragments
2) Change an editor and save it.
Comment 8 Martin Aeschlimann CLA 2007-04-17 03:38:58 EDT
What's an example of a shell title?
Comment 9 Tod Creasey CLA 2007-04-17 07:57:48 EDT
Generally it would be the title of the workbench window. As Phillipe says the best option is to chase down the monitors that have an empty task name.
Comment 10 Martin Aeschlimann CLA 2007-04-17 09:08:28 EDT
I agree it's best to chase down progress monitors with missing task messages.
But I doubt that the shell title is second best. I don't think the window title is of any help here.
What about 'Pending/waiting user operation'?

Note that sentence capitalization is correct here, but not title capitalization. Another reason to not take a shell title...
Comment 11 Tod Creasey CLA 2007-04-17 10:57:28 EDT
That would be fine for me. I just kept the shell title because that is what we used to use.

Philippe what is your opinion?
Comment 12 Philipe Mulet CLA 2007-04-17 22:22:14 EDT
I was thinking of a default label, no shell title; but I am not a UI expert, so cannot break the tie here (pls ask some UI person).

Also, when mentionning 'Pending/Waiting' I was thinking of either 'Pending' or 'Waiting' <g> Pls run this by some native speaker. My claim against 'Blocked' is that it sounded like deadlocked or sthg like that... 
Comment 13 Tod Creasey CLA 2007-04-18 07:43:09 EDT
Waiting would be the better word I would think.

There is no tie to break - Martin and I agree. So if you abstrain we will go with the result of the 2-1-0 vote <grin>.

Reopening to change wording.
Comment 14 Tod Creasey CLA 2007-04-18 09:31:22 EDT
Fixed the String for build >20070418
Comment 15 Tod Creasey CLA 2007-04-30 12:40:51 EDT
Verified in I20070430-0010