Remove the splitter binding

RESOLVED FIXED in Firefox 61

Status

()

RESOLVED FIXED
a year ago
11 months ago

People

(Reporter: bgrins, Assigned: potch)

Tracking

(Blocks: 1 bug)

unspecified
mozilla61
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox61 fixed)

Details

(Whiteboard: [xbl-available])

Attachments

(1 attachment)

This one doesn't do much - we should be able to move splitter.css into components.css and somehow move the [extends] portion out of XBL. https://searchfox.org/mozilla-central/rev/8976abf9cab8eb4661665cc86bd355cd08238011/toolkit/content/widgets/splitter.xml#11-15
Neil, do you have an idea how we can stop relying on `extends="xul:splitter"` in XBL?
Flags: needinfo?(enndeakin)

Comment 2

a year ago
Since the tagname is already splitter, the extends doesn't do anything here and could just be removed.
Flags: needinfo?(enndeakin)
Whiteboard: [xbl-available]
Assignee: nobody → thepotch
Status: NEW → ASSIGNED
(Assignee)

Comment 3

a year ago
Attachment #8959706 - Flags: review?(enndeakin)

Updated

a year ago
Attachment #8959706 - Flags: review?(enndeakin) → review+
Patch just needs reviewer added to the commit message before landing (r=enn)
Keywords: checkin-needed

Comment 7

a year ago
Pushed by shindli@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b42d97021503
Remove the splitter binding r=enn
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/b42d97021503
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Moving to Core:XUL per https://bugzilla.mozilla.org/show_bug.cgi?id=1455336
Component: XP Toolkit/Widgets: XUL → XUL
You need to log in before you can comment on or make changes to this bug.