Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert analyzer z-index changes #1117

Merged
merged 1 commit into from Jun 24, 2021
Merged

Revert analyzer z-index changes #1117

merged 1 commit into from Jun 24, 2021

Conversation

ghys
Copy link
Member

@ghys ghys commented Jun 24, 2021

Reverts the CSS z-index changes made in #1095, without reinstating #1067
& #1068 because now the analyzer route is correctly modal. There doesn't seem to be an
easy way to insert the popup-backdrop between the item picker smart select popup
(or model picker) and the analyzer popup/analyzer controls sheet, especially if there's
another popup behind the analyzer popup with the popup-behind class.

Fixes #1105.

Signed-off-by: Yannick Schaus github@schaus.net

Reverts the CSS z-index changes made in openhab#1095, without reinstating openhab#1067
& openhab#1068 because now analyzer route is
now modal. There doesn't seem to be an
easy way to insert the popup-backdrop
between the item picker smart select popup
(or model picker) and the analyzer popup/
analyzer controls sheet, especially if there's
another popup _behind_ the analyzer popup
with the `popup-behind` class.

Fixes openhab#1105.

Signed-off-by: Yannick Schaus <github@schaus.net>
@ghys ghys requested a review from a team as a code owner June 24, 2021 23:31
@relativeci
Copy link

relativeci bot commented Jun 24, 2021

Job #168: Bundle Size — 10.46MB (-0.01%).

8470044 vs e9b2ded

Changed metrics (4/8)
Metric Current Baseline
Initial JS 1.6MB(~-0.01%) 1.6MB
Initial CSS 602.07KB(~-0.01%) 602.11KB
Cache Invalidation 23.99% 2.68%
Modules 1484(-0.07%) 1485
Changed assets by type (2/7)
            Current     Baseline
CSS 837.21KB (-0.01%) 837.3KB
JS 8.2MB (-0.01%) 8.2MB

View Job #168 report on app.relative-ci.com

@ghys ghys merged commit 6cd28ed into openhab:main Jun 24, 2021
@ghys ghys added the main ui Main UI label Jun 25, 2021
@ghys ghys added this to the 3.1 milestone Jun 25, 2021
@ghys ghys added the bug Something isn't working label Jun 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working main ui Main UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Main UI] Group blocks a analyze graph from being seen.
1 participant