User tests: Successful: Unsuccessful:
Pull Request for Issue #27805
Changes in SCSS as suggested by @infograf768 @brianteeman
Check on any browser
mobile button should not be visible on desktop. gear button should not hide modals/alerts.
mobile button not visible on desktop
Status | New | ⇒ | Pending |
Category | ⇒ | Administration Templates (admin) |
@ravisaxena23 In your description, the expected result and the actual result are the same text. This can't be. Expected result should describe what you expect and how it will be when this PR will be applied, and actual result shall descibe how it is now in J4 without your PR.
@ravisaxena23 In your description, the expected result and the actual result are the same text. This can't be. Expected result should describe what you expect and how it will be when this PR will be applied, and actual result shall descibe how it is now in J4 without your PR.
I will remember it
The only solution I see is
$zindex-mobile-toggle
instead of $zindex-alerts
$zindex-alerts
as it should always be on top of everything.we have now
$zindex-alerts: 1030;
I would do
$zindex-alerts: 10000;
// or more
in /administrator/templates/atum/scss/_variables.scss
Labels |
Added:
?
|
Looks OK now. Weird as it looks like the Alert and the Gear are on the same z-index...
I have tested this item
Looks OK now. Weird as it looks like the Alert and the Gear are on the same z-index...
$zindex-mobile-toggle: 9999; which is present in burger button while gear button hass z index=$zindex-alerts: 1030;
gear has same z-index than alert, but it works fine. I was not speaking of the burger
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-02-06 18:24:12 |
Closed_By | ⇒ | infograf768 | |
Labels |
Added:
?
|
tks
Unfortunately this breaks the toggle at the bottom of the screen.
It should look like this

but with this PR it looks like this
