Bug 344164 - [client] progress links should be more visible
Summary: [client] progress links should be more visible
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Client (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.2   Edit
Assignee: Szymon Brandys CLA
QA Contact:
URL:
Whiteboard:
Keywords: polish
: 349232 (view as bug list)
Depends on:
Blocks: 349296
  Show dependency tree
 
Reported: 2011-04-28 12:43 EDT by Susan McCourt CLA
Modified: 2011-09-01 11:41 EDT (History)
4 users (show)

See Also:
malgorzata.tomczyk: review+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Susan McCourt CLA 2011-04-28 12:43:14 EDT 
we've just added support for links in the progress messages.
The M7 support was done quickly (during the status call)..some improvements

- need to document what HTML can really go in there
- need a special CSS class assigned by the progress support so that the links always appear (vs. just on hover)
- probably need some space between the status icon and html (we insert some for the text)
Comment 1 Tomasz Zarna CLA 2011-06-10 04:19:42 EDT 
I would add there a spinning wheel (or any other kind of spinning widget) so the user knows there is something going on in the background. Dots are not that intuitive. Moreover, if we're getting statuses from more then one task they should be displayed in a drop down list or something similar. This sounds like a separate bug, though.
Comment 2 Susan McCourt CLA 2011-06-10 09:55:15 EDT 
(In reply to comment #1)
> I would add there a spinning wheel (or any other kind of spinning widget) so
> the user knows there is something going on in the background. Dots are not that
> intuitive. 

this bug is for minor cleanup for the 0.2 release, so it's just going to be tweaking what's there.  I had thought about elapsed time, just so there was at least some movement.

> Moreover, if we're getting statuses from more then one task they
> should be displayed in a drop down list or something similar. This sounds like
> a separate bug, though.

yep, that's bug 344012
Comment 3 Szymon Brandys CLA 2011-06-13 03:31:20 EDT 
*** Bug 349131 has been marked as a duplicate of this bug. ***
Comment 4 Susan McCourt CLA 2011-06-13 16:49:06 EDT 
Szymon, now that the header layout has been changed and the progress messages appear on a light color vs. dark, there is no longer a need to use the CSS class "pageActions" on your progress message that has links.  That was the hack I gave you back when we first added this feature.

I removed the class in the live CSS and indeed, you get the nice, blue link that you would expect if no class is specified.  So the progress links look just like other links in the header (good).

So I think this bug is over to you to remove "pageActions" from your progress HTML.
Comment 5 Susan McCourt CLA 2011-06-13 17:40:58 EDT 
*** Bug 349232 has been marked as a duplicate of this bug. ***
Comment 6 Malgorzata Janczarska CLA 2011-06-14 05:24:07 EDT 
looks OK
Comment 7 Szymon Brandys CLA 2011-06-14 05:31:49 EDT 
Fixed with cc9f454e2fa76e7327668eb317323c3ce7503b2b. The link is nice and blue, but I think it should be more visible. I raised Bug 349296 for it.