Closed
Bug 824784
Opened 12 years ago
Closed 10 years ago
Refactor SocialToolbar.updateButton to multiple smaller functions
Categories
(Firefox Graveyard :: SocialAPI, defect)
Firefox Graveyard
SocialAPI
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: jaws, Unassigned)
Details
(In reply to Matthew N. [:MattN] from bug 811835 comment #16) > ::: browser/base/content/browser-social.js > @@ +780,5 @@ > > this.updateButtonHiddenState(); > > let provider = Social.provider; > > let icons = provider.ambientNotificationIcons; > > let iconNames = Object.keys(icons); > > + let socialToolbarItem = document.getElementById("social-toolbar-item"); > > This function is way too large. It should be refactored into more > manageable pieces in a separate bug.
Comment 1•10 years ago
|
||
obsoleted by bug 935640
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Updated•5 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•