Open Bug 1403666 Opened 7 years ago Updated 2 years ago

WebExtension browserActions in the overflow panel need to toggle arrow indicator based on whether they have a subview

Categories

(WebExtensions :: Frontend, enhancement, P5)

Product:
WebExtensions
Component:
Frontend
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: sfoster, Unassigned)

References

(Blocks 1 open bug)

Details

We are using a subviewbutton-nav class to provide the ">" affordance for actions with subviews in the overflow panel. The classList needs to be updated when the panel is shown based on whether they have a popup/subview. Without the class, the subview will still be available, but there's no visible indication for the user that it exists.
I'm not sure we can do this reliably, as extensions can add/change/disable the panel (which would be the subview) at any time. This is especially bad if the ">" affordance makes the toolbar button wider and causes reflow.
This might be better off handled by the front end team dealing with Photon.
Component: WebExtensions: General → WebExtensions: Frontend
Priority: -- → P5
(In reply to Tomislav Jovanovic :zombie from comment #1) > This is especially bad if the ">" affordance makes the toolbar button wider > and causes reflow. It shouldn't do that.
Product: Toolkit → WebExtensions
See Also: → 1407718
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.