?
avatar richard67
richard67
19 Sep 2020

Steps to reproduce the issue

  1. In backend on a desktop screen, reduce browser window width until the admin menu on the left hand side disappears and the burger button is shown at the bottom right corner.
  2. Click on the burger button.
    Result: The admin menu which is now at the bottom of the screen is expanded => ok.
  3. Click again on the burger button.
    Result: The admin menu at the bottom of the screen is collapsed => ok.
  4. Enlarge browser window width until the burger button disappears.
    Result: See section "Actual result" below.

The issue happened to me also sometimes when skipping steps 2 and 3, but only when the particular page was shown for the very first time in that session, and even then not in all cases, so it was hard to reproduce. With steps 2 and 3 I could reproduce it reliably.

There are no Javascript errors shown in the broswer console.

Expected result

The admin menu appears again at the left hand side, preferrably with collapsed status.

Actual result

The admin menu remains invisible.

After a page reload the admin menu appears with collapsed status on the left hand side.

System information (as much as possible)

Clean installation of current 4.0-dev branch, recvent 4.0 nightly build or 4.0 Beta 1.

Additional comments

I've noticed this issue when testing PR #30131 . It happens with and without that PR applied. I think the problem is that in file build/media_source/templates/atum/js/template.es6.js, the setMobile functions sets some CSS class on either the wrapper or the sidebar-wrapper which are not properly reset by the setDesktop function. But I haven't found the error yet.

I have no idea if we have already an issue for that so this here is a duplicate. I haven't found one with a quick search for "burger" and a few other keywords, but I haven't checked deeper.

avatar richard67 richard67 - open - 19 Sep 2020
avatar joomla-cms-bot joomla-cms-bot - change - 19 Sep 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Sep 2020
avatar richard67 richard67 - change - 19 Sep 2020
The description was changed
avatar richard67 richard67 - edited - 19 Sep 2020
avatar richard67 richard67 - change - 19 Sep 2020
The description was changed
avatar richard67 richard67 - edited - 19 Sep 2020
avatar richard67 richard67 - change - 19 Sep 2020
The description was changed
avatar richard67 richard67 - edited - 19 Sep 2020
avatar richard67
richard67 - comment - 19 Sep 2020

I think I have a fix, but I'd like to wait until PR #30131 is merged because otherwise it would need to apply both PR's for testing and later have conflicts after that PR is merged.

avatar richard67 richard67 - change - 19 Sep 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-09-19 10:23:00
Closed_By richard67
avatar richard67
richard67 - comment - 19 Sep 2020

Now as PR #30131 has been merged, I have a solution. I'm not sure if it's the right one, but it works.

Please test PR #30686 and give feedback there. If the PR is wrong, this issue can be re-opened.

avatar richard67 richard67 - close - 19 Sep 2020

Add a Comment

Login with GitHub to post a comment