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

3.1.5: Fehler im CSS-Modul bei Zuweisung von background #6709

Closed
Samson1964 opened this issue Feb 4, 2014 · 10 comments
Closed

3.1.5: Fehler im CSS-Modul bei Zuweisung von background #6709

Samson1964 opened this issue Feb 4, 2014 · 10 comments
Labels
Milestone

Comments

@Samson1964
Copy link

Beispiel 1:
Hintergrundbild zugewiesen, Hintergrundwiederholung zugewiesen
Contao generiert sauberes CSS mit background-image und background-repeat

Beispiel 2:
Hintergrundbild zugewiesen, Hintergrundwiederholung zugewiesen, Hintergrundposition zugewiesen
Contao generiert sauberes, aber falsches CSS nur mit background und den 3 Werten.

Bei Beispiel 2 wird ein zuvor definiertes background-color (in einem anderen Selektor) damit überschrieben. background darf aber erst verwendet werden, wenn eine Hintergrundfarbe bei dieser Bildzuweisung definiert ist!

@leofeyer
Copy link
Member

leofeyer commented Feb 7, 2014

Geht das auch etwas weniger abstrakt? Was genau wurde konfiguriert und was genau ausgegeben?

@Samson1964
Copy link
Author

Okay. Bilder sagen mehr als 1000 Worte.
Beispiel 1 zeigt korrektes CSS:
zwischenablage01
Bei Beispiel 2 ist noch background-position hinzugekommen. Diesmal wird das CSS auf ein nicht korrektes background gekürzt und überschreibt damit ein zuvor in einer anderen Zuweisung definiertes background-color:
zwischenablage02
Ergänzend wäre noch zu sagen, das ich andere Varianten nicht getestet habe, z.B. ob der Farbverlauf auch eine Rolle spielt.

@leofeyer
Copy link
Member

leofeyer commented Feb 8, 2014

Sehe ich nicht als Fehler, da der Stylesheet-Editor ja nicht wissen kann, ob irgendeine andere Formatdefinition bereits eine Hintergrundfarbe definiert. Am besten setzt Du die Farbe an dieser Stelle noch einmal, dann wird sie zur Kurzschreibweise hinzugefügt.

@Samson1964
Copy link
Author

Ich würde an dieser Stelle die Farbe ja setzen, entsprechend Deinem Vorschlag. Allerdings ist das in dem Fall eine globale Farbe, die aus einem Farbschema-CSS kommt, wo zudem die Farben über globale Variablen definiert sind.
Da Contao an meiner Problemstelle den background-color absichtlich mit background auf den Default-Wert zurücksetzt, schreibe ich für diesen Selektor mein CSS in das Feld mit dem eigenen Code. Aber anders gehts halt nicht. Oder würdest Du in purem CSS background benutzen, wenn Du die Hintergrundfarbe auf KEINEN Fall ändern willst? Mit Sicherheit nicht. In Contao machst Du es aber.

@leofeyer leofeyer added Defect and removed Invalid labels Feb 17, 2014
@leofeyer leofeyer added this to the 3.2.8 milestone Feb 17, 2014
@leofeyer
Copy link
Member

Du hast Recht, die Kurzschreibweise darf nur verwendet werden, wenn alle 4 Angaben gemacht wurden.

@leofeyer
Copy link
Member

Behoben in 0917d8c.

@aschempp
Copy link
Member

Ihr solltet bedenken dass einige (z.B. ich immer) davon ausgehen, dass dies so funktioniert. Entsprechend kann es aktuell sein, dass meine Hintergrundfarbe "gelöscht" wird wenn ich ein Bild zuweise, was jetzt nicht mehr der Fall ist! Schlecht für eine Patch-Version :-(

@leofeyer
Copy link
Member

?

@aschempp
Copy link
Member

Wie du weisst verwende ich ausschliesslich den internen Editor. Mir ist bewusst das bei Zuweisung eines Hintergrundbildes mit allen entsprechenden Optionen ein "Short-Syntax" gebaut wird. Entsprechend verwende ich das manchmal auch, um gleichzeitig eine Hintergrundfarbe zu überschreiben. Auch weise ich immer absichtlich eine Background-Position zu auch wenn es die nicht braucht, weil ich weiss dass dann ein Short-Syntax gebaut wird.

Theoretisch wäre es schön, man könnte per Checkbox sagen er soll eine Short-Version bauen. Aber das wäre natürlich wieder widersprüchlich zum bestehenden Verhalten.

@christianromeni
Copy link

Das habe ich mir auch schon ein paar mal gedacht...

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