Closed Bug 408743 Opened 14 years ago Closed 14 years ago

Today-pane close button misses active and hover states on Mac

Categories

(Calendar :: Lightning Only, defect)

PowerPC
macOS
defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: sipaq, Assigned: sipaq)

Details

(Keywords: polish)

Attachments

(1 file, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
The Today-pane close button has active and hover states on Windows, but not on Mac.

I can't test this, since I have no Mac, but it should work.
Attachment #293588 - Flags: review?(Berend.Cornelius)
Attached patch Patch v2Splinter Review
Copy & Paste sucks. This patch should work.
Attachment #293588 - Attachment is obsolete: true
Attachment #293850 - Flags: review?(Berend.Cornelius)
Attachment #293588 - Flags: review?(Berend.Cornelius)
Comment on attachment 293850 [details] [diff] [review]
Patch v2

The patch looks very good to me. Unfortunately I have no Mac available as Daniel took it with himself home, so I have assign the request to him.
Attachment #293850 - Flags: review?(Berend.Cornelius) → review?(daniel.boelzle)
Comment on attachment 293850 [details] [diff] [review]
Patch v2

patch works fine on Mac; r=dbo
Attachment #293850 - Flags: review?(daniel.boelzle) → review+
Keywords: checkin-needed
FIXED on HEAD and MOZILLA_1_8_BRANCH
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
checked in latest lightning build 20080116 -> task is fixed and verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.