Remove support for button[type="menu-button"]

RESOLVED FIXED in Firefox 63

Status

()

P1
normal
RESOLVED FIXED
11 months ago
7 months ago

People

(Reporter: ntim, Assigned: Paolo)

Tracking

(Blocks: 1 bug)

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(3 attachments)

Triage: code cleanup.
Priority: -- → P5
(Assignee)

Updated

7 months ago
Assignee: nobody → paolo.mozmail
Status: NEW → ASSIGNED
Priority: P5 → P1
(Assignee)

Comment 3

7 months ago
The "menu-button" notification button type is unused, and the "menu" type is implemented using a normal button that opens a popup.

Depends on D4529
Depends on: 1487087
Comment on attachment 9004845 [details]
Bug 1457218 - Part 1 - Don't inherit from "menu-button-base" in the "download-subview-toolbarbutton" binding. r=bgrins

Brian Grinstead [:bgrins] has approved the revision.
Attachment #9004845 - Flags: review+
Comment on attachment 9004848 [details]
Bug 1457218 - Part 3 - Remove the "menu-button" binding. r=surkov,bgrins

Brian Grinstead [:bgrins] has approved the revision.
Attachment #9004848 - Flags: review+
Comment on attachment 9004847 [details]
Bug 1457218 - Part 2 - Remove support for menu button types in notification bars. r=bgrins

Brian Grinstead [:bgrins] has approved the revision.
Attachment #9004847 - Flags: review+
Comment on attachment 9004848 [details]
Bug 1457218 - Part 3 - Remove the "menu-button" binding. r=surkov,bgrins

alexander :surkov (:asurkov) has approved the revision.
Attachment #9004848 - Flags: review+

Comment 10

7 months ago
Pushed by paolo.mozmail@amadzone.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ce10788000f5
Part 1 - Don't inherit from "menu-button-base" in the "download-subview-toolbarbutton" binding. r=bgrins
https://hg.mozilla.org/integration/mozilla-inbound/rev/e2d52cbf726d
Part 2 - Remove support for menu button types in notification bars. r=bgrins
https://hg.mozilla.org/integration/mozilla-inbound/rev/6ce3008e2f46
Part 3 - Remove the "menu-button" binding. r=surkov,bgrins

Comment 11

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ce10788000f5
https://hg.mozilla.org/mozilla-central/rev/e2d52cbf726d
https://hg.mozilla.org/mozilla-central/rev/6ce3008e2f46
Status: ASSIGNED → RESOLVED
Last Resolved: 7 months ago
status-firefox63: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.