Closed
Bug 1531119
Opened 6 years ago
Closed 6 years ago
Remove editMenuCommands.inc.xul and make editMenuOverlay.js in charge of constructing the relevant commandset DOM
Categories
(Toolkit :: General, enhancement, P3)
Toolkit
General
Tracking
()
RESOLVED
FIXED
mozilla68
Tracking | Status | |
---|---|---|
firefox68 | --- | fixed |
People
(Reporter: bgrins, Assigned: bgrins)
References
Details
Attachments
(1 file)
This would allow us to load just <script type="application/javascript" src="chrome://global/content/editMenuOverlay.js">
and remove preprocessing for https://searchfox.org/mozilla-central/search?q=editMenuCommands.inc.xul&path=
.
Assignee | ||
Comment 1•6 years ago
|
||
This allows us to drop preprocessing and makes it simpler to add edit menu functionality to any type of document.
Updated•6 years ago
|
Priority: -- → P3
Comment 2•6 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:bgrins, could you have a look please?
Flags: needinfo?(bgrinstead)
Assignee | ||
Comment 3•6 years ago
|
||
(In reply to Release mgmt bot [:sylvestre / :calixte] from comment #2)
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:bgrins, could you have a look please?
Was just waiting until after the merge, will land it shortly.
Flags: needinfo?(bgrinstead)
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Comment 4•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox68:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in
before you can comment on or make changes to this bug.
Description
•