Skip to content

New admin theme vs custom themes? #1164

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

Closed
sreichel opened this issue Aug 23, 2020 · 6 comments
Closed

New admin theme vs custom themes? #1164

sreichel opened this issue Aug 23, 2020 · 6 comments

Comments

@sreichel
Copy link
Contributor

sreichel commented Aug 23, 2020

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

It is just CSS and should break nothing.

@aterjung
Copy link
Contributor

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?

@sreichel
Copy link
Contributor Author

sreichel commented Aug 27, 2020

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
Copy link
Contributor Author

sreichel commented Sep 7, 2020

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

@Flyingmana
Copy link
Contributor

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

@aterjung
Copy link
Contributor

aterjung commented Sep 8, 2020 via email

@kiatng
Copy link
Contributor

kiatng commented Sep 9, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants