Closed Bug 799929 Opened 7 years ago Closed 7 years ago

Downloads Panel item icons are practically invisible on OSX.

Categories

(Firefox :: Downloads Panel, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 19

People

(Reporter: mconley, Assigned: mconley)

References

Details

Attachments

(6 files, 4 obsolete files)

29.98 KB, image/png
Details
26.98 KB, image/png
Details
19.46 KB, image/png
Details
19.88 KB, image/png
Details
2.04 KB, image/png
Details
4.46 KB, patch
mak
: review+
Details | Diff | Splinter Review
Attached image Hard-to-see icons
The icons are white on a gray background, making them very difficult to see.

See screenshot. That can't be right.
Attached image Current icon set (obsolete) —
Here's the current icon set. I think we need something darker. Or, alternatively, we need to make the panel background significantly darker to make the lighter icons visible (though I somehow doubt this is the approach we'll take).
Cc'ing and needinfo'ing shorlander re: icons.
Flags: needinfo?(shorlander)
(In reply to Mike Conley (:mconley) from comment #1)
> Created attachment 669953 [details]
> Current icon set
> 
> Here's the current icon set. I think we need something darker. Or,
> alternatively, we need to make the panel background significantly darker to
> make the lighter icons visible (though I somehow doubt this is the approach
> we'll take).

Yeah these are the old icons from when the panel was still dark. I will make darker icons.
Flags: needinfo?(shorlander)
we basically need all the icons for all the OSes, the current ones must be considered placeholders.
(In reply to Marco Bonardo [:mak] from comment #4)
> we basically need all the icons for all the OSes, the current ones must be
> considered placeholders.

They shouldn't be considered placeholders. They were just updated recently.
From what I remember Paolo used some personally edited icon where they were missing, dunno if you consider that an update, or there was another bug I missed...
(In reply to Marco Bonardo [:mak] from comment #6)
> From what I remember Paolo used some personally edited icon where they were
> missing, dunno if you consider that an update, or there was another bug I
> missed...

Paolo put the new ones I gave him in the last patch of Bug 760607 I think :)
(In reply to Stephen Horlander from comment #7)
> Paolo put the new ones I gave him in the last patch of Bug 760607 I think :)

Hm, Stephen, when I landed the patch I asked you if the icons were still visible
on the white background... you lied ;-)

Seriously, the new icons landed basically at the same time as the change of
panel background, so this was kind of a known issue, though I expected the
icons to be more visible than the screenshot shows.

We might consider quick "invert" on the PNG while we wait for the final icons.
(In reply to Paolo Amadini [:paolo] from comment #8)
> (In reply to Stephen Horlander from comment #7)
> > Paolo put the new ones I gave him in the last patch of Bug 760607 I think :)
> 
> Hm, Stephen, when I landed the patch I asked you if the icons were still
> visible
> on the white background... you lied ;-)

I think I said they were kind of visible ;)
Attached patch Patch v1 (obsolete) — Splinter Review
Binary diff to replace the original buttons.png.

I'll throw up a screenshot or two and then r?.
Assignee: nobody → mconley
Hovering over these swaps in the darker icon, and click-holding gives us the blue icons, as expected.
I notice that for completed downloads, hovered download items have a blue background. The dark icons on that blue background are difficult to see.
Comment on attachment 670131 [details] [diff] [review]
Patch v1

Stephen:

I know you said that I just needed r+ on this, but the background-is-blue-on-hover thing makes me wonder if we should do one of the following:

1) Not change the background colour of a downloaded item to blue on hover
2) Swap out the icon for a fourth one - a lighter one - when the item is hovered. This would necessitate several new icons.

How should it appear?

-Mike
Attachment #670131 - Flags: ui-review?(shorlander)
We do have a fourth row of inverted "open in folder" icons on Windows XP and
sometimes on Vista, for that specific case.

While we're here, let's confirm that we're not going to reduce the hovering area
again in the short term, as suggested in bug 787285 (I'm experimenting with the
new needinfo flag to ask for Stephen's feedback on that).
we can fix the icons when bug 787285 will be fixed, I don't feel like that's a blocker so we could just think about it later.
(In reply to Paolo Amadini [:paolo] from comment #16)
> We do have a fourth row of inverted "open in folder" icons on Windows XP and
> sometimes on Vista, for that specific case.
> 
> While we're here, let's confirm that we're not going to reduce the hovering
> area
> again in the short term, as suggested in bug 787285 (I'm experimenting with
> the
> new needinfo flag to ask for Stephen's feedback on that).

The brighter blue color was actually supposed to be for a selected state that we don't have :) I don't know that we even need a hover state on OSX but if we keep then it should be something more subtle.

In regards to bug 787285 I don't think the solution, if one is required, will be to go back to the clipped item area.
Combined the new icons and the inverted icons for the hover state.
Attachment #670127 - Attachment is obsolete: true
Actually until we change the hover state we will only need the inverted hover icons. So I just inserted one column of the dark icons.
Attachment #670384 - Attachment is obsolete: true
Comment on attachment 670131 [details] [diff] [review]
Patch v1

Sorry, I should have included the inverted icons with the updated dark icons. Until we figure out the entire hover state appearance it just needs to use the dark icons in the non-hover state.
Attachment #670131 - Flags: ui-review?(shorlander) → ui-review-
Attached patch Patch v2Splinter Review
This patch does the following:

- Eliminates the downloadCancel and downloadRetry hover / active rules on OSX until bug 787285 is resolved.
- Uses the light icons for downloadShow when hovering the download item. The hover and active states still work for this button, but this all might be academic if we end up moving that button outside of the hoverable-download-item area.

So, basically, this is a visibility stopgap for OSX until bug 787285 gets sorted out.
Attachment #669953 - Attachment is obsolete: true
Attachment #670131 - Attachment is obsolete: true
Attachment #670444 - Flags: ui-review?(shorlander)
Comment on attachment 670444 [details] [diff] [review]
Patch v2

Got the thumbs up from shorlander over IRC, so clearing ui-r?.
Attachment #670444 - Flags: ui-review?(shorlander) → review?(mak77)
Comment on attachment 670444 [details] [diff] [review]
Patch v2

Review of attachment 670444 [details] [diff] [review]:
-----------------------------------------------------------------

<3 academic patches
Attachment #670444 - Flags: review?(mak77) → review+
https://hg.mozilla.org/mozilla-central/rev/64bf0d8480a5
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
You need to log in before you can comment on or make changes to this bug.