Closed
Bug 1218780
Opened 9 years ago
Closed 9 years ago
Notifications shouldn't use a hand cursor
Categories
(Toolkit :: Themes, defect)
Toolkit
Themes
Tracking
()
RESOLVED
FIXED
mozilla45
People
(Reporter: phlsa, Assigned: jaws)
References
Details
Attachments
(1 file)
707 bytes,
patch
|
MattN
:
review+
ritu
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
When moving the mouse over a notification, the cursor should remain the default arrow cursor.
Flags: needinfo?(jaws)
Assignee | ||
Comment 1•9 years ago
|
||
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Flags: needinfo?(jaws)
Attachment #8679446 -
Flags: review?(MattN+bmo)
Assignee | ||
Updated•9 years ago
|
Component: DOM: Push Notifications → Themes
Product: Core → Toolkit
Updated•9 years ago
|
Attachment #8679446 -
Flags: review?(MattN+bmo) → review+
Comment 3•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/302277d37f39
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Comment 4•9 years ago
|
||
Comment on attachment 8679446 [details] [diff] [review] Patch Approval Request Comment [Feature/regressing bug #]: Notification improvements in Fx44 [User impact if declined]: Hand cursor appears on notifications which doesn't match most platform conventions [Describe test coverage new/current, TreeHerder]: m-c [Risks and why]: Trivial, low risk [String/UUID change made/needed]: None
Attachment #8679446 -
Flags: approval-mozilla-aurora?
Comment 5•9 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/94dca3a1d0d1
status-firefox44:
--- → fixed
Comment on attachment 8679446 [details] [diff] [review] Patch Seems like a low risk fix, let's uplift to Aurora44.
Attachment #8679446 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
I just noticed (from comment 5) that this was uplifted to Aurora44 already. Comment 6 doesn't mean much except a rubber stamp. :)
Comment 8•9 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/302277d37f39
status-b2g-v2.5:
--- → fixed
Comment 9•9 years ago
|
||
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
status-b2g-v2.5:
fixed → ---
Comment 10•9 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/94dca3a1d0d1
status-b2g-v2.5:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•