Closed Bug 1005677 Opened 11 years ago Closed 10 years ago

Pocket / reading list icon breaks panel layout

Categories

(WebExtensions :: General, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: alexandre.mbm, Unassigned)

References

Details

Attachments

(3 files)

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release) Build ID: 20140428194100
Blocks: 960258
Status: UNCONFIRMED → NEW
Component: Untriaged → Add-ons
Ever confirmed: true
Product: Firefox → Tech Evangelism
Summary: Disorganized icons on panel of the newer Firefox → Pocket / reading list icon breaks panel layout
Version: 29 Branch → Trunk
Jorge, do you know if we have contacts for the pocket developers, and/or if the pocket part of this is a duplicate? Alexandre: all the issues are with add-on buttons, as far as I can tell from the screenshot. There's not a lot Firefox can do about the issues those buttons are causing. :-(
Flags: needinfo?(jorge)
The red words "None" were painted by me. The red characters '?' were painted by me.
When I empty the panel...
(In reply to Alexandre Magno from comment #4) > Created attachment 8418307 [details] > firefox-panel-bug-empty.png > > When I empty the panel... It looks like there is still something in there, like a spacer or separator. What does running the following in the browser console (ctrl+shift+j) return: CustomizableUI.getWidgetIdsInArea("PanelUI-contents").join(',') ?
Flags: needinfo?(alexandre.mbm)
< CustomizableUI.getWidgetIdsInArea("PanelUI-contents").join(',') > "customizableui-special-spacer15"
Flags: needinfo?(alexandre.mbm)
(In reply to Alexandre Magno from comment #6) > < CustomizableUI.getWidgetIdsInArea("PanelUI-contents").join(',') > > "customizableui-special-spacer15" Grand. You can fix it by running: CustomizableUI.removeWidgetFromArea("customizableui-special-spacer15"); We're tracking fixing this generally in bug 1003588.
OK. Thanks. Part of the problem was resolved. The Pocket continues wrong...
I installed Pocket and didn't see any problems, having it in the main toolbar or the menu panel. How was Pocket identified as the problem here, given the rather large amount of add-on icons I see in the screenshots?
Flags: needinfo?(jorge)
(In reply to Jorge Villalobos [:jorgev] from comment #9) > I installed Pocket and didn't see any problems, having it in the main > toolbar or the menu panel. How was Pocket identified as the problem here, > given the rather large amount of add-on icons I see in the screenshots? Because in attachment 8417062 [details], it is clearly wider than the other icons, and causing the developer tools to shift right, and the third icon to no longer fit. It's possible that a pocket update resolved this, or that the issue is specific to a certain OS / font size.
> How was Pocket identified as the problem here, given the rather large amount of add-on icons I see in the screenshots? When it is added at panel, the panel breaks the sequence. But it isn't I who rewritten the bug's title.
The shape of the Pocket's button is unlike any other.
Ah, I see now. I moved it to the one place where it didn't break the layout. I'll notify the developers.
Jorge - Can we close this bug now?
Flags: needinfo?(jorge)
Yes, I think this was fixed long ago.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(jorge)
Resolution: --- → FIXED
Component: Add-ons → General
Product: Tech Evangelism → WebExtensions
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: