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

Unterstützung für Größensuffixe bei Alignments #26

Merged
merged 2 commits into from Apr 8, 2019
Merged

Unterstützung für Größensuffixe bei Alignments #26

merged 2 commits into from Apr 8, 2019

Conversation

@RoflCopter24
Copy link
Contributor

@RoflCopter24 RoflCopter24 commented Apr 5, 2019

Im Backend kann man bei 'Gridgrößen und -spalten' bei jeder Größe 'Vertikale Ausrichtung' auswählen.
Bei der Ausgabe wird dies jedoch nicht berücksichtigt, es wird für alle Größen immer nur mehrfach zum Beispiel align-self-end anstatt align-self-xx-end ausgegeben.

Ich habe die entsprechenden Funktionen ergänzt, sodass nun die Größen wie md, lg usw. richtig ausgegeben werden sofern sie vorhanden sind.

Vorher:
ce_bs_gridSeparator col-12 align-self-center col-lg-6 align-self-end
Nachher:
ce_bs_gridSeparator col-12 align-self-center col-lg-6 align-self-lg-end

Im Backend kann man bei 'Gridgrößen und -spalten' bei jeder Größe 'Vertikale Ausrichtung' auswählen.
Bei der Ausgabe wird dies jedoch nicht berücksichtigt, es wird für alle Größen immer nur 'align-self-end' anstatt 'align-self-xx-end' ausgegeben.

Ich habe die entsprechenden Funktionen ergänzt, sodass nun die Größen wie md, lg usw. richtig ausgegeben werden sofern sie vorhanden sind.
@dmolineus dmolineus self-assigned this Apr 5, 2019
@dmolineus dmolineus added this to In progress in Contao Bootstrap v2 via automation Apr 5, 2019
@dmolineus dmolineus added this to the 2.1.0 milestone Apr 5, 2019
@dmolineus
Copy link
Member

@dmolineus dmolineus commented Apr 5, 2019

Nice, I somehow missed that theses classes are also size specific available.

This project uses the contao-community-alliance coding standards which requires that each author is listed in the file header. Besides some alignments in the docs does not fit the coding standard. Would you please address this issues? You can check them local by running a composer update an ant. Or you can have a look at the Travis CI build logs.

@RoflCopter24
Copy link
Contributor Author

@RoflCopter24 RoflCopter24 commented Apr 6, 2019

Sure! The header has been added and phpcbf fixed some spaces.

@dmolineus dmolineus changed the base branch from master to develop Apr 8, 2019
@dmolineus dmolineus changed the base branch from develop to master Apr 8, 2019
@dmolineus dmolineus changed the base branch from master to develop Apr 8, 2019
Copy link
Member

@dmolineus dmolineus left a comment

Thanks, LGTM. Gonna be released with version 2.1.0 following semantic versioning.

@dmolineus dmolineus merged commit d380bed into contao-bootstrap:develop Apr 8, 2019
1 check passed
Contao Bootstrap v2 automation moved this from In progress to Done Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants