1148911 - Nontransparent download progress bar in OS X Yosemite

Status: RESOLVED DUPLICATE of bug 1386046

(Core :: Widget: Cocoa, defect, P3)

Description

[Kuba]

Attachment: progressbar.png

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:36.0) Gecko/20100101 Firefox/36.0
Build ID: 20150320202338

Steps to reproduce:

Start downloading a file


Actual results:

The progress bar has white opaque background (see attached screenshot), tested on a fresh Yosemite install with fresh Firefox profile, also tested with Nightly 39.0a1.


Expected results:

The border around the progress bar should be transparent.

Comment 1

[Gustavo Vera Scuderi]

Attachment: nontransparent-progressbar.png

It happens to me too. Firefox 37.0.2 on OS X 10.10.3 Yosemite.

Seems like a bug since progress bars on Yosemite have less height. On Mavericks looks fine.

Comment 2

[Gustavo Vera Scuderi]

Attachment: The progress bar with the older bounds removed

Finally looked at the code and figured out what happened here.

The rect made for the progress bar is still using the pre-Yosemite size, so the new Yosemite styled progress bar aligns itself right in the middle, revealing all the empty space around. This only happens on Yosemite, of course. On Mavericks looks fine.

After a while I (kinda) fixed it. Will post a patch as soon as I clean up the code. However, the progress bar still looks weird considering the new height on Yosemite and given that maybe Apple won't change it in a long time. Maybe we need a new kind of download indicator like the Chrome "download pie".

Comment 3

[Gustavo Vera Scuderi]

Attachment: Proposed patch v1

This is the first solution I've found. It's not that elegant, but it gets the job done for now.

It uses AppKit for detecting the OS X version at runtime. If it's Mavericks or pre-Mavericks doesn't do anything to what it is now, but if it is Yosemite it will use a different height for the progress bar background.

Comment 4

[Steven Michaud [:smichaud] (Retired)]

I won't be able to get to this review right away.  With luck it'll be sometime next week.

Comment 6

[Markus Stange [:mstange]]

Comment on attachment 8607341 [details] [diff] [review]
Proposed patch v1

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

::: widget/cocoa/nsMacDockSupport.mm
@@ +156,3 @@
>      [mProgressBackground lockFocus];
> +    [[NSColor clearColor] set];
> +    NSRectFill(NSRectFromCGRect(mProgressBoundsMask));

Why did you choose not to clear the whole mProgressBounds?

I think just changing "whiteColor" to "clearColor" should be all that's necessary to fix this bug.

Comment 7

[Gustavo Vera Scuderi]

Just putting clearColor doesn't do, it just makes a clear transparent space (clipping the logo) instead of the white space we see here: https://bugzilla.mozilla.org/attachment.cgi?id=8596204. That's why I choose to reduce the bounds of it.

Comment 8

[Gustavo Vera Scuderi]

However, now that you mention it, I'll try to skip the drawing of the mProgressBounds space if it's Yosemite. Might post another patch later :)

Comment 9

[Markus Stange [:mstange]]

Oh, I see the problem now. Can we store the icon without the progress bar in a different buffer, so that we can always recomposite both original images? Then we don't have to worry about getting the bounds of the progress bar right.

Comment 10

[Gustavo Vera Scuderi]

I think it might be possible to remove the white space drawing altogether. The redraw function for the progress bar just need the mProgressBounds rect values.

Something I noticed last night is that the white background seems to be just for make the downloading progress bar more noticeable. Just look at the result of my fix using clearColor: https://bugzilla.mozilla.org/attachment.cgi?id=8607327. With whiteColor it would be better, but we'll have to clip again to make it have round corners.

Again, I'll try removing the rect drawing when I get home. I guess it would be a better solution than guessing the bounds.

Comment 11

[Valter Vicente]

Any news on this? Almost a year later since the last comment (with a nice screenshot!) I still get that progress bar inside a white rectangle.

Comment 12

[lothar.haeger]

Still unresolved on 52.0 ESR. Would be great if this could be solved or - in case this is not possible short term - if an option to disable the progress bar on the dock icon be added. I do not find it all that useful personally and rather don't see it at all than in it's current form...

Comment 13

[lothar.haeger]

Oh, how about replacing the progress bar with a native macOS feature, those little numbers in a red circle that you get e.g. when Mail has unread messages? Clear the counter when the download window has been opened after the download completed, maybe...

Comment 14

[Stephen A Pohl [:spohl]]

I was not aware of this bug and this has been fixed in bug 1386046. If there is any additional work required we should file new followup bugs. Closing as dupe of bug 1386046.