Closed
Bug 1470361
Opened 6 years ago
Closed 6 years ago
Remove non-override stylesheet support for nsIEditorStyleSheets.
Categories
(Core :: DOM: Editor, enhancement)
Core
DOM: Editor
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Nobody uses them (c-c nor bluegriffon), and they add complexity because they're doc stylesheets but they're not owned by the document.
This bug will only remove support from editor, style system simplifications will come in a followup bug.
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8986965 [details]
Bug 1470361: Remove nsIEditorStyleSheets.addStyleSheet / removeStyleSheet / replaceStyleSheet.
https://reviewboard.mozilla.org/r/252200/#review258678
Is editor/libeditor/tests/test_bug468353.html still necessary? This test calls removeStyleSheet method. It is OK if removing this test.
Assignee | ||
Comment 3•6 years ago
|
||
You're right, I missed that one. I removed the test. Thanks!
Comment hidden (mozreview-request) |
Comment 5•6 years ago
|
||
mozreview-review |
Comment on attachment 8986965 [details]
Bug 1470361: Remove nsIEditorStyleSheets.addStyleSheet / removeStyleSheet / replaceStyleSheet.
https://reviewboard.mozilla.org/r/252200/#review259072
Attachment #8986965 -
Flags: review?(m_kato) → review+
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/mozilla-inbound/rev/913161ea522e
Remove nsIEditorStyleSheets.addStyleSheet / removeStyleSheet / replaceStyleSheet. r=m_kato
Comment 7•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Updated•6 years ago
|
Assignee: nobody → emilio
Updated•5 years ago
|
Blocks: redesign-editor-scriptable-API
You need to log in
before you can comment on or make changes to this bug.
Description
•