? ? Pending

User tests: Successful: Unsuccessful:

avatar ravisaxena23
ravisaxena23
5 Feb 2020

Pull Request for Issue #27805

Summary of Changes

Changes in SCSS as suggested by @infograf768 @brianteeman
Screenshot (43)

Testing Instructions

Check on any browser

Expected result

mobile button should not be visible on desktop. gear button should not hide modals/alerts.

Actual result

mobile button not visible on desktop

Documentation Changes Required

avatar ravisaxena23 ravisaxena23 - open - 5 Feb 2020
avatar ravisaxena23 ravisaxena23 - change - 5 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Feb 2020
Category Administration Templates (admin)
avatar ravisaxena23 ravisaxena23 - change - 5 Feb 2020
The description was changed
avatar ravisaxena23 ravisaxena23 - edited - 5 Feb 2020
avatar ravisaxena23 ravisaxena23 - change - 5 Feb 2020
The description was changed
avatar ravisaxena23 ravisaxena23 - edited - 5 Feb 2020
avatar brianteeman
brianteeman - comment - 5 Feb 2020

Unfortunately this breaks the toggle at the bottom of the screen.

It should look like this
image

but with this PR it looks like this
image

avatar richard67
richard67 - comment - 5 Feb 2020

@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.

avatar ravisaxena23
ravisaxena23 - comment - 6 Feb 2020

@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

avatar ravisaxena23
ravisaxena23 - comment - 6 Feb 2020

Unfortunately this breaks the toggle at the bottom of the screen.

It should look like this
image

but with this PR it looks like this
image

This is due to z-index I will fix it. I have a question why this blue bar not come on my laptop what I'm doing is wrong.?

avatar infograf768
infograf768 - comment - 6 Feb 2020

The only solution I see is

  1. Use again $zindex-mobile-toggle instead of $zindex-alerts
  2. Modify the variable for $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

avatar ravisaxena23 ravisaxena23 - change - 6 Feb 2020
Labels Added: ?
avatar ravisaxena23 ravisaxena23 - change - 6 Feb 2020
The description was changed
avatar ravisaxena23 ravisaxena23 - edited - 6 Feb 2020
avatar infograf768
infograf768 - comment - 6 Feb 2020

Looks OK now. Weird as it looks like the Alert and the Gear are on the same z-index...

avatar infograf768 infograf768 - test_item - 6 Feb 2020 - Tested successfully
avatar infograf768
infograf768 - comment - 6 Feb 2020

I have tested this item successfully on b86a807


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27818.

avatar ravisaxena23
ravisaxena23 - comment - 6 Feb 2020

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;

avatar infograf768
infograf768 - comment - 6 Feb 2020

gear has same z-index than alert, but it works fine. I was not speaking of the burger

avatar infograf768
infograf768 - comment - 6 Feb 2020

@Quy can you test?

avatar Quy Quy - test_item - 6 Feb 2020 - Tested successfully
avatar Quy
Quy - comment - 6 Feb 2020

I have tested this item successfully on b86a807


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27818.

avatar Quy Quy - change - 6 Feb 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 6 Feb 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27818.

avatar infograf768 infograf768 - change - 6 Feb 2020
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: ?
avatar infograf768 infograf768 - close - 6 Feb 2020
avatar infograf768 infograf768 - merge - 6 Feb 2020
avatar infograf768
infograf768 - comment - 6 Feb 2020

tks

Add a Comment

Login with GitHub to post a comment