Closed
Bug 799749
Opened 12 years ago
Closed 12 years ago
Add back the Show Desktop Notifications menuitem for the Social API toolbar menu
Categories
(Firefox Graveyard :: SocialAPI, defect)
Tracking
(firefox17+ verified, firefox18+ verified)
VERIFIED
FIXED
Firefox 19
People
(Reporter: jaws, Assigned: jaws)
References
Details
(Keywords: regression, Whiteboard: [Fx17])
Attachments
(1 file, 1 obsolete file)
8.43 KB,
patch
|
jaws
:
review+
Gavin
:
approval-mozilla-aurora+
Gavin
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
The patch for bug 790112 accidentally removed the Show Desktop Notifications toolbar menuitem. This patch adds it back, along with a test to make sure this doesn't happen again. This affects Firefox 17 and 18 as well, but the patch is minimal and doesn't introduce any strings. The strings weren't removed on accident, just the menuitem.
Attachment #669779 -
Flags: review?(mhammond)
Comment 1•12 years ago
|
||
Comment on attachment 669779 [details] [diff] [review] Patch It would be even better if the test checked the menuitems did something :)
Attachment #669779 -
Flags: review?(mhammond) → review+
Assignee | ||
Comment 2•12 years ago
|
||
Also fixed the toolbar icons in the other tests. Ok'd by Mark on IRC, 5:08 PM <jaws> markh: ok for me to fix the toolbar icons when i push the menuitem fix? 5:09 PM <markh> fine with me :) 5:10 PM <jaws> i mean the toolbar icons in the tests of course 5:10 PM <jaws> they're currently using 48x48 icons, which doesn't seem right The tests don't check the "checked" state toggling due to bug 799761.
Attachment #669779 -
Attachment is obsolete: true
Attachment #669810 -
Flags: review+
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/88bb0e2f0373
Flags: in-testsuite+
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/88bb0e2f0373
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
Updated•12 years ago
|
Updated•12 years ago
|
Attachment #669810 -
Flags: approval-mozilla-beta+
Attachment #669810 -
Flags: approval-mozilla-aurora+
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/663a97922297 https://hg.mozilla.org/releases/mozilla-aurora/rev/24d2399d1ca1
Verified fixed across all branches.
Updated•5 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•