Closed Bug 553482 Opened 14 years ago Closed 14 years ago

Add icons for buttons in download progress XBL binding

Categories

(Toolkit :: Add-ons Manager, defect, P2)

defect

Tracking

()

VERIFIED FIXED
mozilla1.9.3a5
Tracking Status
blocking2.0 --- final+

People

(Reporter: Unfocused, Assigned: Unfocused)

References

Details

(Whiteboard: [rewrite])

Attachments

(1 file, 1 obsolete file)

The buttons in the download progress don't have any icons - need to add some.
Priority: -- → P1
Assignee: nobody → bmcbride
Keywords: uiwanted
Version: unspecified → Trunk
Attached file Add-ons manager download icons (obsolete) —
Attached are download progress icons in a variety of states - anything else needed here?
Priority: P1 → P2
I'll take ui-wanted down for now.  Final graphics are still needed on this and others, but that should probably be a separate bug
Keywords: uiwanted
Is this done?
(In reply to comment #3)
> Is this done?

No.
(In reply to comment #2)
> I'll take ui-wanted down for now.  Final graphics are still needed on this and
> others, but that should probably be a separate bug

Jennifer, shall we file a separate bug or what is your intention? I don't want that we forget about those icons. Why not attach the icons to that bug? Putting back uiwanted until we have a solution.
blocking2.0: --- → ?
Keywords: uiwanted
Attached patch PatchSplinter Review
Attachment #434967 - Attachment is obsolete: true
Attachment #448338 - Flags: review?(dtownsend)
(In reply to comment #5)
> Jennifer, shall we file a separate bug or what is your intention? I don't want
> that we forget about those icons. Why not attach the icons to that bug? Putting
> back uiwanted until we have a solution.

Yea, all that will be in some separate bug(s) - its pretty much a separate project (since it involved all in-content pages, not just about:addons). I don't think anything has been filed yet - I believe shorlander will be filing once things are ready. And the change in visual styling seems to be significant enough that these won't be left forgotten.
Status: NEW → ASSIGNED
Keywords: uiwanted
Attachment #448338 - Flags: review?(dtownsend) → review+
blocking2.0: ? → final+
(In reply to comment #6)
> Created an attachment (id=448338) [details]
> Patch

Pushed as: http://hg.mozilla.org/mozilla-central/rev/e079f587a584
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Verified fixed with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.3a5pre) Gecko/20100605 Minefield/3.7a5pre
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Flags: in-litmus-
Target Milestone: --- → mozilla1.9.3a5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: