magento-lts icon indicating copy to clipboard operation
magento-lts copied to clipboard

New admin theme vs custom themes?

Open sreichel opened this issue 5 years ago • 6 comments

Summary (*)

I've tested the admin theme (#1008) only with vanilla LTS and it worked like a charm. After updating some shops to latest LTS i had to realize that all shops made changes to admin templates. Instead of changing default templates, they added a new theme that is based on default ... (seems correct to me)

    <stores>
        <admin>
            <design>
                <package>
                    <name>default</name>
                </package>
                <theme>
                    <default>default</default>
                    <template>some_name</template>
                </theme>
            </design>
        </admin>
    </stores>

So "openmage admin theme" is never loaded ...

  • it shows default login page (default templates loaded)
  • only loads default css

Examples (*)

See above.

Proposed solution

  • add "openmage" to fallback logic? NO! (???)

or

  • replace default admin theme with #1008
  • remove admin theme switcher

It is just CSS and should break nothing.

sreichel avatar Aug 23 '20 22:08 sreichel

ok, this behavior is intended to be shure custom themes are not affected. The shops i am supporting all use the default theme so i did not see this as a problem. Did you check if the themes would break if you change the base theme to openmage?

aterjung avatar Aug 27 '20 20:08 aterjung

Did you check if the themes would break if you change the base theme to openmage?

Yes, i did. It shows openmage theme, but the theme-switcher does not work anymore.

sreichel avatar Aug 27 '20 23:08 sreichel

Do we still need the default theme, or can we (@aterjung) replace it? Actually it does not work with common (?) template rewrites.

sreichel avatar Sep 07 '20 22:09 sreichel

We can not replace it for backwards compatibility reasons, we could just copy it, which would increase the needed maintenance

Flyingmana avatar Sep 08 '20 06:09 Flyingmana

I am not shure if I understand the Problem! If there is a custom theme, isn’t it right to ignore the openmage theme? Is the Problem that the theme switch dose not switch to openmage theme then? Is this really a problem?

Diplom Wirtschaftsinformatiker (FH)

Am 08.09.2020 um 00:04 schrieb sv3n [email protected]:

 Do we still need the default theme, or can we (@aterjung) replace it? Actually it does not work with common (?) template rewrites.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

aterjung avatar Sep 08 '20 21:09 aterjung

@aterjung An example to the problem: I have a custom template in app\design\adminhtml\default\ihe\template\sales\order\view\history.phtml, where I set my custom theme called ihe, where I added the admin user name to show which comment is made by who. Now, if I want to use the openmage theme, how do I fallback to it?

As suggested by @sreichel, one way is to replace the default theme with openmage.

Another way is make a new package for openmage, I actually prefer this, for infinite theme fallback.

kiatng avatar Sep 09 '20 03:09 kiatng