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

meta theme-static-root is also used in admin #2756

Open
freesoul opened this issue Jul 20, 2022 · 1 comment
Open

meta theme-static-root is also used in admin #2756

freesoul opened this issue Jul 20, 2022 · 1 comment

Comments

@freesoul
Copy link

This meta tag should be present in the template.html of the admin, or template.html should be also overrided by the one by the template in the admin.

The reason is that this variable is used for assets in widgets from the theme (example reuse.png in reuses widget from french theme)

@maudetes
Copy link
Contributor

Thank you for this ticket and the one in udata-front. I suggest that we continue the discussion here since both are similar.

I see two things here.
First, we don't have any more placeholder widgets in udata (reuse.png, organization.png, etc.), thus having an issue with this widget with udata in standalone in any case.
Secondly, metadata.html is not overridden by theme (gouvfr for example) in admin. But I wonder if we should actually override it, since the admin is out of scope from udata-front. In any case, we probably have too much duplicate code between both metadata.html leading to confusion and wonder if we could use a hook logic for additional metadata in front for example instead. Would be interested in your opinions @abulte, @quaxsze.

Seeing that the second point won't probably be fixed very soon and we would like a udata standalone to work nicely in any case , I would suggest to replace theme_static code in udata by static_root, adding missing assets in udata core. We wouldn't be able to override these with a theme though. Would that be a working solution for you @freesoul?

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

No branches or pull requests

2 participants