Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

CSS .ce_gallery .cols_x #7138

Closed
Metis77 opened this issue Jun 30, 2014 · 5 comments
Closed

CSS .ce_gallery .cols_x #7138

Metis77 opened this issue Jun 30, 2014 · 5 comments
Labels
Milestone

Comments

@Metis77
Copy link

Metis77 commented Jun 30, 2014

.cols_x zu .ce_gallery anfügen

Da Editoren die Anzahl der Spalten einer Galerie variieren dürfen und sollen, sollte die Spaltenzahl dem ContentElement als Klasse mitgegeben werden.

Warum?

Responsive Design

Gerade im Responsive Design ist es von Vorteil, weil oft zB eine 5 spaltige Galerie anders umbrechen soll als eine 3 Spaltige.

High Res Display (Retina Images)

Ein beliebter Ansatz, der auch in vielen Contao Themes so verwendet wird, basiert darauf, dass die Galerie Bilder skaliert werden, und deshalb nicht mehr die px Angaben des <img> wiederspiegeln.
Daher ergibt sich das Layout nicht mehr aus der wirklichen Breite der Bilder, sondern zB aus .image_container { width: 33%} für eine dreispaltige Galerie.
Wenn in solch einem Theme der Editor die Spaltenzahl ändert, ohne die vom theme vorgesehene Klasse anzugeben, zerhaut es das Design.

Deshalb schlage ich vor, dem .ce_gallery noch ein .cols_x mitzugeben.

@leofeyer
Copy link
Member

leofeyer commented Jul 3, 2014

Wie lässt sich das initiale Problem reproduzieren?

@Metis77
Copy link
Author

Metis77 commented Jul 3, 2014

@leofeyer Ich weiß jetzt nicht, wie ich das anders beschreiben soll.
Es handelt sich hierbei um keinen Bug, eher um einen Verbesserungsvorschlag.

Vielleicht hilft ein Beispiel:
http://demo.rocksolidthemes.com/?theme=contao-tao&language=de
Dort Features -> "Bilder & Galerien" wählen.

Bei der Galerie mit fünf Bildern pro Reihe die Klasse -per-row-5 entfernen und man hat das von mir beschriebene Verhalten.

Da oft Galerien abhängig von der Spaltenanzahl gestyled werden, sollte Anzahl der Spalten direkt als Klasse cols_xausgegeben werden.

@fjacobi
Copy link

fjacobi commented Jul 4, 2014

+1 als Feature Request

@leofeyer leofeyer added this to the 3.4.0 milestone Jul 16, 2014
@fritzmg
Copy link
Contributor

fritzmg commented Jul 21, 2014

+1, kann aber jeder selbst ganz einfach machen. gallery_default:

<ul class="row-<?php echo $this->perRow; ?>">

@leofeyer
Copy link
Member

Hinzugefügt in 7c9e6c7.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants