Closed
Bug 1178078
Opened 10 years ago
Closed 7 years ago
Don't hard-coding CSS variable name directly in panelUI.xml
Categories
(WebExtensions :: General, defect)
WebExtensions
General
Tracking
(firefox41 affected, firefox42 affected, firefox43 affected, firefox44 affected)
People
(Reporter: magicp.jp, Unassigned)
Details
Attachments
(3 files)
User Agent: Mozilla/5.0 (Windows NT 6.3; rv:41.0) Gecko/20100101 Firefox/41.0
Build ID: 20150627030211
Steps to reproduce:
Please see the attached file "PanelUI.png"
1) Click Youtube High Definition button
2) Click Developer button
Actual results:
Please see the attached file "PanelUI.png"
A) subView-arrow is not shown
B) Overlap subViews
Expected results:
It happened, if don't use the:root pseudo-class "--panel-ui-exit-subview-gutter-width" in PanelUIOverlay.css. Because "--panel-ui-exit-subview-gutter-width" is hardcoded in panelUI.xml.
Component: Untriaged → Theme
OS: Unspecified → All
Hardware: Unspecified → All
This problem also has occurred on the following builds.
- Nightly 42.0a1 (20150629134017)
- Firefox Developer Edition 41.0b2 (20150629134017)
Comment 2•10 years ago
|
||
This appears to be a bug with the YouTube High Definition add-on that you have installed. In the second screenshot their button is still in the :active state, even though the Developer Tools subview is showing. The YouTube High Definition add-on should be closing their panel when a click outside of the panel occurs.
Baris, can you look in to fixing this in your add-on? You can look at the consumeoutsideclicks attribute.
Component: Theme → Add-ons
Flags: needinfo?(barisderin)
Product: Firefox → Tech Evangelism
Version: 41 Branch → Firefox 41
> This appears to be a bug with the YouTube High Definition add-on that you
> have installed. In the second screenshot their button is still in the
> :active state, even though the Developer Tools subview is showing. The
> YouTube High Definition add-on should be closing their panel when a click
> outside of the panel occurs.
Jared, I upload a new image "identity-popup-multiView.png" I believe that this bug is occurred by "--panel-ui-exit-subview-gutter-width" Please check css and xul.
I added modified "panelUIOverlay.css" for reproduction.
Modified...
Line 22: comment out
Line 114: replace "var(--panel-ui-exit-subview-gutter-width)" with "38px"
Line 1055: replace "var(--panel-ui-exit-subview-gutter-width)" with "38px"
Line 1072: replace "var(--panel-ui-exit-subview-gutter-width)" with "38px"
Line 1079: replace "var(--panel-ui-exit-subview-gutter-width)" with "38px"
Comment 6•10 years ago
|
||
I'm not sure what this is showing? If you disable the YouTube High Definition add-on does the issue go away? If so, then the bug is caused by the add-on and the error will need to be fixed there.
>It happened, if don't use the:root pseudo-class "--panel-ui-exit-subview-gutter-width" in >PanelUIOverlay.css. Because "--panel-ui-exit-subview-gutter-width" is hardcoded in >panelUI.xml.
As I wrote, these issues occur when "--panel-ui-exit-subview-gutter-width" is not used. the YouTube High Definition popup is just example for overlap. If use "--panel-ui-exit-subview-gutter-width", overlap is not occur(the YouTube High Definition popup is hidden). The subView-arrow is not shown, regardless of the YouTube High Definition is installed or not.
Comment 9•10 years ago
|
||
I'm sorry, it's still hard for me to understand what you're saying. It sounds like there is some part of the CSS in our tree that is not using this CSS variable --panel-ui-exit-subview-gutter-width? What part isn't using it that should be using it?
Flags: needinfo?(jaws)
Reporter | ||
Comment 10•10 years ago
|
||
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #9)
> I'm sorry, it's still hard for me to understand what you're saying. It
> sounds like there is some part of the CSS in our tree that is not using this
> CSS variable --panel-ui-exit-subview-gutter-width? What part isn't using it
> that should be using it?
CSS Variables should be entities defined by theme developer or user. But, in this case CSS variable name is hard coded in panelUI.xml. It's difficult for us to find this and modify.
I think hard coding should not be use in this situation. If there is not alternative way, should have a comment and specific variable naming in css.
Comment 11•10 years ago
|
||
I don't know how we could achieve this other ways. What is your recommendation? Matt, do you have an opinion here?
Flags: needinfo?(magicp.jp)
Flags: needinfo?(jaws)
Flags: needinfo?(MattN+bmo)
Reporter | ||
Comment 12•10 years ago
|
||
Comment 13•9 years ago
|
||
Closing old needinfo requests...
Flags: needinfo?(barisderin)
Flags: needinfo?(MattN+bmo)
Status: UNCONFIRMED → NEW
status-firefox41:
--- → affected
status-firefox42:
--- → affected
status-firefox43:
--- → affected
status-firefox44:
--- → affected
Ever confirmed: true
Summary: Overlap subViews and subView-arrow is not shown in PanelUI-popup → Don't hard-coding CSS variable name directly in panelUI.xml
Version: Firefox 41 → unspecified
Comment 14•7 years ago
|
||
Mass-closing bugs that relate to legacy versions of add-ons or are otherwise no longer worth tracking. Please comment if you think this bug should be reopened.
Sorry for the bugspam. Made you look, though!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•6 years ago
|
Component: Add-ons → General
Product: Tech Evangelism → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•