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

Fehlende 'Bootstrap Navigation Classes' Ausgabe #12

Closed
RoflCopter24 opened this issue Nov 27, 2018 · 2 comments
Closed

Fehlende 'Bootstrap Navigation Classes' Ausgabe #12

RoflCopter24 opened this issue Nov 27, 2018 · 2 comments

Comments

@RoflCopter24
Copy link

@RoflCopter24 RoflCopter24 commented Nov 27, 2018

Moin!

Kleine Fehlerkorrektur:
Das im Backendmodul Navigationsmenü ausfüllbare Feld Bootstrap Navigation Classes wird nicht ausgegeben.

In src/Helper/NavigationHelper.php nach Zeile 72 fehlt:

$attributes->addClass($this->template->navClass);

Dann flutscht das :-)

@dmolineus dmolineus self-assigned this Nov 28, 2018
@dmolineus
Copy link
Member

@dmolineus dmolineus commented Nov 28, 2018

Ja, das ist verwirrend und der Modularisierung geschuldet. Die Funktion stammt von contao-bootstrap/templates und wird im Template nav_bs berücksichtigt. Ist contao-bootstrap/templates nicht installiert, gibt es das Feld nicht.

Ich werde voraussichtlich den Support wie vorgeschlagen einbauen. Wenn die Template-Komponente nicht installiert ist, sollten ja keine Probleme auftreten - nur die Funktion fehlt halt.

@dmolineus dmolineus added this to To do in Contao Bootstrap v2 via automation Nov 28, 2018
@dmolineus dmolineus added this to the 2.1.3 milestone Nov 28, 2018
@dmolineus dmolineus modified the milestones: 2.1.3, 2.1.4 Jun 11, 2019
dmolineus added a commit that referenced this issue Jun 11, 2019
Add navClass if it's defined in the template
(Feature of contao-bootstrap/templates)
@dmolineus
Copy link
Member

@dmolineus dmolineus commented Jun 11, 2019

Implemented with ef1beaf.

@dmolineus dmolineus closed this Jun 11, 2019
Contao Bootstrap v2 automation moved this from To do to Done Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants