Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Weird artifacts under the progress bar

RESOLVED FIXED in mozilla15

Status

()

Core
Widget: Cocoa
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: reuben, Assigned: reuben)

Tracking

Trunk
mozilla15
All
Mac OS X
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=mstange][lang=c++])

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
Created attachment 620543 [details]
Screen recording

A weird line and a few black pixels randomly appear under the progress bar on pinstripe. See attachment. (Please ignore the parrot and the pegleg.)
I think this happens because the progress bar doesn't report its overflow in nsNativeThemeCocoa::GetWidgetOverflow.
Component: Downloads Panel → Widget: Cocoa
Product: Firefox → Core
QA Contact: downloads.panel → cocoa
Whiteboard: [good first bug][mentor=mstange][lang=c++]

Comment 2

5 years ago
Got a link to the relevant source?
Sure, the only occurrence of "nsNativeThemeCocoa::GetWidgetOverflow" in the source: http://mxr.mozilla.org/mozilla-central/source/widget/cocoa/nsNativeThemeCocoa.mm#2387
What should be the value of the overflow here? Currently the code inside nsNativeThemeCocoa::GetWidgetOverflow is about "focus ring" but AFAICT, there is no focus ring for the progress element.
It looks like the white 1px shadow under the progress bar is the only part that overflows, but one should test different sizes and states, maybe there's also overflow to the other sides. But as a first try I'd just extend the the overflow rect's height by 1 pixel (and not add the focus ring margin for progress bars).
(Assignee)

Comment 6

5 years ago
Created attachment 621144 [details] [diff] [review]
Patch

(In reply to Markus Stange from comment #1)
> I think this happens because the progress bar doesn't report its overflow in
> nsNativeThemeCocoa::GetWidgetOverflow.

Interesting. The shadow is 2 pixels high.
Attachment #621144 - Flags: review?(mstange)
Comment on attachment 621144 [details] [diff] [review]
Patch

Great!
Attachment #621144 - Flags: review?(mstange) → review+
(Assignee)

Updated

5 years ago
Assignee: nobody → reuben.morais
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/dbc45d173470
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/dbc45d173470
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.