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

css-grid for rocksolid-columns #17

Closed
mlwebworker opened this issue Feb 21, 2019 · 6 comments
Closed

css-grid for rocksolid-columns #17

mlwebworker opened this issue Feb 21, 2019 · 6 comments

Comments

@mlwebworker
Copy link

Ich würde gerne css-grid bei den rocksolid-columns einsetzen. Dafür fehlen mir aber Klassen beim Spaltenumschlag (ce_rs_columns_start.html5), die sich aus der Gesamtanzahl der Spalten für die jeweilige Breite (Mobile, Tablet, Desktop) ergeben. Ist so etwas machbar?
Ich hatte schon mal in den Code reingeschaut, habe aber keine Idee wie ich es anstellen soll, dass die mit $media und $columnsTotal generierten Klassen auch im Template ce_rscolumns_start.html5 ausgegeben werden.

@ausi
Copy link
Member

ausi commented Feb 21, 2019

Im Spaltenumschlag-Template kannst du über MadeYourDay\RockSolidColumns\Element\ColumnsStart::getColumnsConfiguration($this->arrData) auf die Klassen-Konfiguration zugreifen. Reicht das für deinen Anwendungsfall aus?

@mlwebworker
Copy link
Author

Das sollte reichen. Ich werde mal schauen, dass ich mich da als Nicht-Programmierer durchbeisse.

@mlwebworker
Copy link
Author

mlwebworker commented Feb 22, 2019

Es hat mich zwar ein paar Stunden gekostet um mir zu erarbeiten wie man das Array zerpflückt damit ich an genau die Klassen rankomme die ich brauche. Aber letzten Endes mit Erfolg und einem gewissen Lerneffekt. Vielleicht kannst Du mir noch sagen, ob man das auch eleganter lösen könnte.
Hier mein angepasstes Template ce_rs_columns_start.html

<?php 
	$config = MadeYourDay\RockSolidColumns\Element\ColumnsStart::getColumnsConfiguration($this->arrData);
	$classes = $config["large"][0][0] . " " . $config["medium"][0][0] . " " . $config["small"][0][0];
?>`
`<div class="rs-columns <?php echo $classes ?> <?php echo $this->class ?>"<?php echo $this->cssID ?><?php if ($this->style): ?> style="<?php echo $this->style ?>"<?php endif ?>>
	<?php if ($this->headline): ?>
		<<?php echo $this->hl ?>><?php echo $this->headline ?></<?php echo $this->hl ?>>
	<?php endif ?>

@ausi
Copy link
Member

ausi commented Feb 22, 2019

Die Anpassung sieht in Ordnung aus 👍

@mlwebworker
Copy link
Author

Danke. Hast Du grundsätzlich Interesse an den Anpassungen auf css-grid?

@ausi
Copy link
Member

ausi commented Feb 25, 2019

Bei einer Umstellung der Erweiterung auf CSS-Grid, wäre es sinnvoll auch die Struktur der Spaltenumschläge so umzugestalten, dass die Vorteile von CSS-Grid besser genutzt werden können.

Mit dem aktuellen Aufbau werden wir vermutlich kein CSS-Grid einsetzen.

@ausi ausi closed this as completed Apr 11, 2019
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