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

Falsche (?) CSS-Anweisung im Seitenheader: #main .inside #5170

Closed
NinaG opened this issue Dec 20, 2012 · 13 comments

Comments

@NinaG
Copy link

commented Dec 20, 2012

Contao schreibt bei aktivem Layoutbuilder + Grid (Seitenaufbau mit Spalte links + Main) automatisch diese CSS-Anweisung in den Seitenheader:

main .inside { margin-left: 30px; margin-right: 10px; }

Diese Anweisung verhindert aber, dass in der Hauptspalte die Grids sauber funktionieren, da ihnen schlicht und einfach Platz fehlt (durch die abgezogenen Margins).

Wie kommt es zu dieser Anweisung bzw. was ist ihr Zweck? Also wie sollen Grids weiterhin funktionieren, obwohl diese Anweisung da steht? Oder haben wir da irgendwas übersehen?

@pmmueller

This comment has been minimized.

Copy link

commented Dec 20, 2012

Hier noch der Link zum Forumbeitrag, in dem das etwas ausführlicher diskutiert wird:
https://community.contao.org/de/showthread.php?36862-Problem-sobald-ein-Padding-gesetzt-wird

@NinaG

This comment has been minimized.

Copy link
Author

commented Dec 20, 2012

Ich konnte das in der Contao-Demo auch gerade nachstellen.

@leofeyer

This comment has been minimized.

Copy link
Member

commented Dec 20, 2012

da ihnen schlicht und einfach Platz fehlt (durch die abgezogenen Margins)

Wie viel Platz fehlt denn?

@leofeyer

This comment has been minimized.

Copy link
Member

commented Dec 20, 2012

Hab mir gerade eine Testumgebung gebaut und #wrapper, #left .inside und #main .inside einen Hintergrund zugewiesen. Die Aufteilung stimmt schon mal.

Bildschirmfoto 2012-12-20 um 15 37 55

Was genau ist das Problem? Müsste der Text in der Hauptspalte ganz links stehen?

@leofeyer

This comment has been minimized.

Copy link
Member

commented Dec 20, 2012

Das margin-left sollte wohl nur 20px sein, oder?

leofeyer added a commit that referenced this issue Dec 20, 2012

@leofeyer

This comment has been minimized.

Copy link
Member

commented Dec 20, 2012

Behoben in 3310f98. Ganz so trivial war es dann doch nicht, aber jetzt sollte es passen:

Bildschirmfoto 2012-12-20 um 16 15 53

@leofeyer leofeyer closed this Dec 20, 2012

leofeyer added a commit that referenced this issue Dec 27, 2012

@leofeyer

This comment has been minimized.

Copy link
Member

commented Dec 27, 2012

Sorry, musste noch mal geändert werden. Der Schlüssel ist, dass die linke bzw. rechte Spalte eine Breite von 240px und nicht 220px erhält. Dann braucht es nämlich gar keine Anpassungen (siehe 110dc1c)!

@BugBuster1701

This comment has been minimized.

Copy link
Contributor

commented Dec 28, 2012

Zu Deutsch, nicht auf die gewünschte grid class Breite sondern auf die grid class Breite noch 2 mal Abstand dazu nehmen. (hier also 220px + 2 x10px)

@aportmann

This comment has been minimized.

Copy link

commented Jan 8, 2013

wenn ich mir die screenshots anschaue, dann stimmts doch an etwas grundlegendem nicht. während bei left und right der text am grid korrekt klebt, gibt es beim content ein padding. ergo stimmt das grid nicht, sobald die hg-faren weggelassen werden. oder mache ich einen denkfehler?

@leofeyer

This comment has been minimized.

Copy link
Member

commented Jan 8, 2013

Das ist korrekt. Die Module in den Seitenspalten haben noch kein Margin (keine Grid-Klassen). Ich setze momentan Themes um mit dem CSS-Framework und kann keine Fehler mehr in der aktuellen Implementierung feststellen.

@aportmann

This comment has been minimized.

Copy link

commented Jan 8, 2013

mh, ein grid im grid dürfte aber nach aussen keine margins mehr beinhalten. sonst steht ja nichts auf dem grid sondern alles um den margin des inneren grids verschoben.

nach meinem verständnis dürfte in content kein abstand sein, sonst steht der inhalt, sofern nicht farblich hinterlegt, nicht im grid.

@aportmann

This comment has been minimized.

Copy link

commented Jan 9, 2013

hat in 3.03 nochmals etwas geändert? nun stimmt zwar der textanfang jeweils auf das grid, dafür klebt #main links am rand und recht an right. #right dementsprechend am rechten rand und links an #main.

das beispiel von leo oben zeit allerdings ein anderes verhalten. wobei ich das verhalten in 3.03 korrekt finde, wollte es nur noch hier aufs trapez bringen.

screen

@Zwergenmeister

This comment has been minimized.

Copy link

commented Jan 16, 2013

@aportmann mit 3.0.3 läuft bei mir jetzt alles normal wie gewollt im zwei spalten Layout, ist nichts mehr mit #main margin-left. Hatte die Problem auch mit 3.0.1 im Zusammenhang mit dem Layout Builder + res Grid (ohne @media)

tobaco added a commit to tobaco/core that referenced this issue Jan 21, 2013

tobaco added a commit to tobaco/core that referenced this issue Jan 21, 2013

leofeyer added a commit that referenced this issue Oct 31, 2014

leofeyer added a commit that referenced this issue Oct 31, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.