Closed Bug 930845 Opened 8 years ago Closed 3 years ago
Remove unused "expander" binding and expander
As far as I can tell, no Firefox code uses it. But it's been in the toolkit forever. There's no documentation on MDN about it, either. It might be the least supported widget (and that's saying something)... Really, we should see if it's a candidate for removal as dead code. Or we should support it with documentation and a mochitest.
I still don't see any usage either, so I believe it can be removed: https://searchfox.org/mozilla-central/source/toolkit/content/xul.css#1159 https://searchfox.org/mozilla-central/source/toolkit/content/widgets/expander.xml https://searchfox.org/mozilla-central/search?q=expander
Whiteboard: [xbl-remove-unused] → [xbl-remove-unused][xbl-available]
Comment on attachment 8931855 [details] [diff] [review] Removed expander binding and expander.xml You should also remove expander.css
Comment on attachment 8931855 [details] [diff] [review] Removed expander binding and expander.xml See above comment.
Attachment #8931855 - Flags: review?(enndeakin) → review-
86ecce74, are you still working on this?
Attachment #8931855 - Attachment is obsolete: true
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/be780bf2c08a Removed expander binding, expander.css and expander.xml; r=enn
You need to log in before you can comment on or make changes to this bug.