Closed Bug 854555 Opened 8 years ago Closed 8 years ago

icons in the Downloads view are slightly "squashed" on OS X

Categories

(Firefox :: Bookmarks & History, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 23

People

(Reporter: jfkthame, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

Attachments

(6 files, 4 obsolete files)

In the Downloads view of the Library window (e.g. accessed by clicking Show All Downloads), the downloaded item icons are slightly squashed vertically (or fattened sideways, if you prefer to think of it that way).

This is most obvious for icons that include circular elements, such as the Nightly globe or the QuickTime logo in the attached screenshot.
Component: Bookmarks & History → Downloads Panel
This appears to be a result of reducing the height of richlistitem.download in bug 823095; the new height of 6em is not quite large enough for the 64px icons we're getting, so they get scaled down in the y-direction to make them fit.

Accordingly, the simplest fix is to increase the height of those items.
Blocks: 823095
Component: Downloads Panel → Bookmarks & History
Attachment #729170 - Flags: review?(mconley)
Assignee: nobody → jfkthame
Personally, however, I think the icons in that view look way too big, and it'd be better to constrain them to 32px and reduce the item height accordingly; something like this. But that's a much more drastic change; maybe UX people really want the big icons there?
Attachment #729175 - Flags: review?(mconley)
(In reply to Jonathan Kew (:jfkthame) from comment #1)
> This appears to be a result of reducing the height of richlistitem.download
> in bug 823095; the new height of 6em is not quite large enough for the 64px
> icons we're getting, so they get scaled down in the y-direction to make them
> fit.
> 
> Accordingly, the simplest fix is to increase the height of those items.

Can we define the height using -moz-max(Xpx,7em), or some such, to make things more robust to changes in default font sizes (which IIRC affects em sizing but not px sizing)?
(In reply to :Gavin Sharp (use gavin@gavinsharp.com for email) from comment #4)
> Can we define the height using -moz-max(Xpx,7em), or some such, to make
> things more robust to changes in default font sizes (which IIRC affects em
> sizing but not px sizing)?

Also the panel may have similar problems (http://mxr.mozilla.org/mozilla-central/source/browser/themes/windows/downloads/downloads.css#54)
Fwiw, we reduced the height in the Library to increase number of entries visible at once, I'd probably vote for the smallest em increase we may take.

Btw, it's possible UX wants to evaluate changes to the icons size, we may try asking.
Using -moz-max() as suggested in comment 4 should make this a bit more robust.
Attachment #729233 - Flags: review?(mconley)
Attachment #729170 - Attachment is obsolete: true
Attachment #729170 - Flags: review?(mconley)
(In reply to Marco Bonardo [:mak] (Away Mar 27 - Apr 2) from comment #5)
> Btw, it's possible UX wants to evaluate changes to the icons size, we may
> try asking.

I agree. Needinfo'ing shorlander.
Flags: needinfo?(shorlander)
For comparison, here's a screenshot of the Downloads view using the "smaller icons" patch, where the download item icons are 32x32 px (and the item height is reduced to 4em). To my eye, this seems more in proportion to the rest of the interface (as well as allowing more items to be visible in the list).
Comment on attachment 729549 [details]
screenshot with 32x32 download icons

What do you think of this shorlander?
Attachment #729549 - Flags: ui-review?(shorlander)
(In reply to Mike Conley (:mconley) from comment #9)
> Comment on attachment 729549 [details]
> screenshot with 32x32 download icons
> 
> What do you think of this shorlander?

the screenshot should also show an in-progress download to be able to evaluate properly... that would add a third row with the progress bar.
In this case, the presence of the progress bar causes the in-progress item to expand vertically - note that it is slightly taller than the other items in the list. It will automatically shrink to the standard height when the download completes.

Personally, I don't think this would bother me; it seems quite reasonable for the "active" item(s) to be enlarged a bit while they have a progress indicator present. But I could understand if it's considered undesirable.
Here's an intermediate option: use 48px icons, and a correspondingly larger item height (but still slightly smaller than the current height). This allows room for the progress indicator to be displayed without altering the item height in the list.
Comment on attachment 735105 [details]
screenshot, download manager with 48px icons

Could we get some guidance on what to do with the icons here, please? The current "squashed" icons (attachment 729168 [details]) look really poor, but which fix is preferred: increase the item height to avoid squashing them, or use smaller (48px or 32px) icons?
Attachment #735105 - Flags: ui-review?(shorlander)
Since users are already complaining about items taking too much space, I'd probably vote to avoid increasing the items height.
FWIW, I agree - the 64px icons seem excessively big to me, and they're the factor forcing such a large item height.

I'll attach the "medium" version of the patch, which makes the icons 48px (as shown in attachment 735105 [details]). Maybe we can get that reviewed and landed...
Attachment #740749 - Flags: review?(mconley)
This is only a problem in the Library view on Retina displays as far as I can tell. 

The icons in the panel and the Library view are (and are supposed to be) 32 x 32 at regular resolution. So 32 is the right size.
Flags: needinfo?(shorlander)
Comment on attachment 735105 [details]
screenshot, download manager with 48px icons

The icon should be 32px we just need to use the correct @2x icon on Retina displays. The panel doesn't appear to have this problem.
Attachment #735105 - Flags: ui-review?(shorlander) → ui-review-
Comment on attachment 729549 [details]
screenshot with 32x32 download icons

Cancelling, see other review comment.
Attachment #729549 - Flags: ui-review?(shorlander)
OK, in that case I believe this is all we need to do here.
Attachment #741370 - Flags: review?(shorlander)
Attachment #729175 - Attachment is obsolete: true
Attachment #729175 - Flags: review?(mconley)
Can I assume based on these latest developments that my review is no longer required on attachment 729299 [details] [diff] [review] and attachment 740749 [details] [diff] [review]?
Flags: needinfo?(jfkthame)
Attachment #729233 - Attachment is obsolete: true
Attachment #729233 - Flags: review?(mconley)
Attachment #740749 - Attachment is obsolete: true
Attachment #740749 - Flags: review?(mconley)
Indeed; I've obsoleted those. (Though if you want to take the review on attachment 741370 [details] [diff] [review], based on shorlander's comment re the desired result, I expect that'd be fine.)
Flags: needinfo?(jfkthame)
Comment on attachment 741370 [details] [diff] [review]
ensure download icons are 32px even on Retina Macs

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

Tested on regular and Retina, looks good!
Attachment #741370 - Flags: review?(shorlander) → review+
Blocks: osx-hidpi
https://hg.mozilla.org/mozilla-central/rev/a1ead1605ade
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:23.0) Gecko/20130426 Firefox/23.0

Verified as fixed on latest Nightly (buildID: 20130426030834).
You need to log in before you can comment on or make changes to this bug.