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

Openmage admin theme: Change background-color for active sections #4219

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

sreichel
Copy link
Contributor

@sreichel sreichel commented Sep 22, 2024

Description (*)

Lightgrey background for active sections.

No changes for legacy theme. Dont want to work with plain CSS files.

Maybe someone wants to add SCSS-files for it .... there are some online converters, that do it.

Fixed Issues (if relevant)

  1. Fixes Admin theme: missing CSS for active links/sections  #4207

@github-actions github-actions bot added Template : admin Relates to admin template Component: Adminhtml Relates to Mage_Adminhtml labels Sep 22, 2024
@addison74
Copy link
Contributor

I am not using the OpenMage Backend theme but I will give it a try. Please post a few screenshots to understand what is wrong and how looks the fix.

@sreichel
Copy link
Contributor Author

Neither in legacy nor in openmage theme active section /menu is highlighted. There is only a hover-effect.

After this PR the active section/menu is not white, its lightgrey.

over

@addison74
Copy link
Contributor

It works as expected for the OpenMage theme.

For Magento legacy theme there was a discussion here #3452 (comment).

@sreichel
Copy link
Contributor Author

For legacy theme, please add SCSS files ;)

Here you go https://jsonformatter.org/css-to-scss ...

@kiatng kiatng merged commit 34647f2 into OpenMage:main Sep 23, 2024
1 check passed
@sreichel sreichel deleted the fixes-4207 branch September 23, 2024 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Adminhtml Relates to Mage_Adminhtml Template : admin Relates to admin template
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Admin theme: missing CSS for active links/sections
3 participants