I have tested this item ? unsuccessfully on 5c4b3e1

When the sidebar is collapsed you now can't see the flyout menu items

Beside the above, the PR also has SCSS code style errors reported here: https://ci.joomla.org/joomla/joomla-cms/43224/1/20 :

> stylelint --config build/.stylelintrc.json -s scss "administrator/components/com_media/resources/**/*.scss" "administrator/templates/**/*.scss" "build/media_source/**/*.scss" "templates/**/*.scss" "installation/template/**/*.scss"


administrator/templates/atum/scss/blocks/_sidebar.scss
 12:5  ✖  Expected "top" to come before "overflow"   order/properties-order

I have tested this item ? unsuccessfully on 5c4b3e1

When the sidebar is collapsed you now can't see the flyout menu items

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

Right, It's needed to check when the sidebar is closed, so we don't have to hide the flyouts.
This is working for me.

#wrapper:not(.closed) .sidebar-wrapper .sidebar-sticky {
    overflow: hidden;
}

I try to add this in the SCSS.

@simbus82 And still the linter compains about the same as before:

administrator/templates/atum/scss/blocks/_sidebar.scss
 8:3  ✖  Expected "overflow" to come before "box-shadow"   order/properties-order

So please fix as advised here by the linter.

Thanks in advance.

I have found a better way!
(Also because it is not easy to understand in which scsss file I should have put the style for the "#wrapper:not(.closed) .sidebar-wrapper .sidebar-sticky").

.sidebar-wrapper with overflow: hidden, and when ".closed" with overflow: visibile, so the flyout menu can be visible.

If there are no other "hidden" functions, I also tried it from mobile and it doesn't give any problems.

Could you kindly test if it works well?

@simbus82 And still the linter compains about the same as before:

administrator/templates/atum/scss/blocks/_sidebar.scss
 8:3  ✖  Expected "overflow" to come before "box-shadow"   order/properties-order

So please fix as advised here by the linter.

@richard67 sorry, i don't use a linter.

@richard67 sorry, i don't use a linter.

@simbus82 At the bottom of the PR you can find the failed tests and follow the links there which at the end lead to the logile showing that. And for your convenience I pasted it here. What else can I do to make you to fix it? It tells you lind number and file and what to fix.

@richard67 sorry, i don't use a linter.

@simbus82 At the bottom of the PR you can find the failed tests and follow the links there which at the end lead to the logile showing that. And for your convenience I pasted it here. What else can I do to make you to fix it? It tells you lind number and file and what to fix.

I was apologizing for not using it in the process, sorry! I'm used to alphabetical order ?

administrator/templates/atum/scss/blocks/_sidebar.scss

8 | 7:3 ✖ Expected "overflow" to come before "background-color" order/properties-order

other than the code style issue above this works great

I have tested this item ✅ successfully on e75125f

Works as described.

I have tested this item ✅ successfully on e75125f

RTC

Thanks!

Thanks to you guys!