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

Classes added to form container template causing issues #610

Open
ag-tim opened this issue Sep 6, 2019 · 0 comments
Open

Classes added to form container template causing issues #610

ag-tim opened this issue Sep 6, 2019 · 0 comments

Comments

@ag-tim
Copy link

ag-tim commented Sep 6, 2019

From the comment in "https://github.com/phase2/particle/blob/master/apps/drupal/templates/form/container.html.twig" it sounds like the two classes were added to this override template for purposes of the site search form. The 'mx-sm-3' class in particular is causing problems because this template is called in many places, some of which are not even forms. Adding spacing to all of these different places that the template is used is, I think, too broad a brush and they should probably be removed from the template and added differently for the site search case.
It's a pretty simple fix, I'm just not sure what other issues changing it may cause.

Drupal admin toolbar:
BEGIN OUTPUT from 'core/themes/stable/templates/form/container.html.twig
div class="toolbar-menu-administration

From views:
BEGIN OUTPUT from 'core/themes/stable/templates/form/container.html.twig
div class="views-element-container"

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

1 participant