search

FriendsOfREDAXO / slice_columns

SliceColumns is a REDAXO AddOn. Grid arrangement for REDAXO article slices.
MIT License
22 stars 2 forks source link

Gedanken zum HTML im Frontend #14

Open tbaddade opened 3 years ago

tbaddade commented 3 years ago

Ich nutze hier die Voreinstellungen des AddOns:

Im Frontend kommt um jeden Slice ein div.col-sm-X. Das finde ich verwirrend, da ich bisher nur 2 Slices im Backend angepasst habe. Andere nicht bearbeitete Slices erhalten immer automatisch ein div.col-sm-12.

Entfernt man dann in den Einstellungen das 12er, erscheinen im Frontend Notices

Notice: Undefined offset: 12 in redaxo/src/addons/slice_columns/lib/columns.php on line 110

und das div verliert die CSS-Klasse.

Bildschirmfoto 2021-09-30 um 19 51 06

Wie definiert man das Wrapper div? Bei Bootstrap wäre es das div.row.

PS: Danke für das AddOn und für eure Zeit die ihr hinein gesteckt habt.

skerbis commented 3 years ago

Das Wrapper Div habe ich bei mir direkt im Template. Das AddOn erstellt keine Wrapper

tbaddade commented 3 years ago

Das Wrapper Div habe ich bei mir direkt im Template.

Dann sind aber alle Slices des Artikels automatisch mit im Wrapper. Will man nicht mal öfters die row öffnen und schließen? Auch um andere Elemente dazwischen darzustellen. Deshalb auch oben der Gedanke, dass nicht um jeden Slice diese div.col-sm ausgegeben wird.

skerbis commented 3 years ago

Ja es muss Definitionen für jede Spaltenbreite geben. Die CSS legst du aber selber fest, also du kannst es auch leer lassen.

row / section Das mache ich in meinem Fall durch ein Modul. Das die Row / Section unterbricht und dann wieder öffnet. Aktuell kann dabei ein leeres DIV entstehen, wenn man direkt mit einer neuen Section / Row beginnt. Lässt sich aber sicher noch abfangen.

Bestimmte Module kann man ja auch ausschließen. Diese bekommen kein Wrapper-DIV.

Vielleicht fällt uns ja noch was elegantes ein. Im Moment weiß ich nicht wie. An der Stelle kann man ja nur Blöcke setzen.

abra100pro commented 2 years ago

Ich habe da verschiedene Varianten versucht – komm' aber nicht drauf. Hast Du ein Code-Beispiel?

Lässt sich sowas erreichen?

section div.container div.row div.col ... /div.row /div.container /section

Neicul commented 11 months ago

Wäre super wenn man das generierte HTML verändern könnte, in meinem Fall bräuchte ich auch mindestens eine Row

skerbis commented 11 months ago

Ich weiß nicht wie man ohne extra Modul ein Wrapper wie bei dir eine row mit diesem AddOn oder überhaupt in der Struktur öffnen und schließen könnte. Man will ja nicht nur eine Row öffnen sondern diesen noch Eigenschaften mitgeben wie Hintergründe oder ähnliches. @Neicul Dafür nutze ich immer ein Modul das im Grunde eine Row öffnet, wann immer sie benötigt wird.