Closed Bug 407063 Opened 17 years ago Closed 17 years ago

Bottom border of title bar on Leopard does not dim properly

Categories

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

x86
macOS
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: cbarrett, Assigned: cbarrett)

References

Details

Attachments

(1 file)

Attached patch fix v1.0Splinter Review
STR: - Focus a Firefox window. - Click on another window. Expected results: Titlebar and bottom border of it dim Actual results: Titlebar dims, bottom border does not. This is my fault, I somehow missed writing this code for bug 404096, and the testcase I put in the bug for QA didn't say "and make sure it matches TextEdit" or anything. Patch attached, has some extra comment fixes as well.
Attachment #291774 - Flags: review?(joshmoz)
Attachment #291774 - Flags: superreview?(roc)
Attachment #291774 - Flags: review?(joshmoz)
Attachment #291774 - Flags: review+
Attachment #291774 - Flags: superreview?(roc) → superreview+
Flags: blocking1.9+
Priority: -- → P3
Comment on attachment 291774 [details] [diff] [review] fix v1.0 This is a small patch to fix a small visual glitch in the titlebar. It certainly annoys me when using nightlies that exhibit this behavior. Low risk.
Attachment #291774 - Flags: approvalM10?
Comment on attachment 291774 [details] [diff] [review] fix v1.0 We'll get this in the next release. aM10-.
Attachment #291774 - Flags: approvalM10? → approvalM10-
Checking in widget/src/cocoa/nsCocoaWindow.mm; /cvsroot/mozilla/widget/src/cocoa/nsCocoaWindow.mm,v <-- nsCocoaWindow.mm new revision: 1.126; previous revision: 1.125 done
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: