Closed Bug 965560 Opened 7 years ago Closed 7 years ago
sidebar toolbarbutton shows even when there is no sidebar to show
[aurora] The toolbar button shows when you install a provider that does not have a sidebar. One such provider uses only a status button, in which case two buttons appear in the toolbar, one for the status panel, one for the sidebar. Following patch prevents showing the sidebar toolbarbutton if no sidebar providers exist.
Assignee: nobody → mixedpuppy
Attachment #8368921 - Flags: review?(felipc)
Attachment #8368921 - Flags: review?(felipc) → review+
Comment on attachment 8368921 [details] [diff] [review] only show button when we have sidebar providers Assuming this doesn't beat uplift, would need beta approval instead...basically need this in fx28. [Approval Request Comment] Bug caused by (feature/regressing bug #): socialapi toolbarbutton User impact if declined: a second misleading button icon for users of some providers. Testing completed (on m-c, etc.): aurora fx28 Risk to taking this patch (and alternatives if risky): low String or IDL/UUID changes made by this patch: none
Since it hasn't landed on central yet, might need to land post-merge, I'll carry over the approval nom if necessary and tracking to ensure this lands in 28.
Comment on attachment 8368921 [details] [diff] [review] only show button when we have sidebar providers Cf comment #4 from Lukas and since the merge has been done, aurora does not need the uplift while it is now needed for the beta.
You need to log in before you can comment on or make changes to this bug.