terminal42 / contao-pageimage

MIT License
12 stars 14 forks source link

Improve HEAD CSS #37

Closed Aybee closed 5 years ago

Aybee commented 5 years ago

Hallo Andreas, im Template mod_background_image https://github.com/terminal42/contao-pageimage/blob/master/templates/mod_background_image.html5#L3-L9 hast du dieses CSS.

html {
  background: url("<?php echo specialchars($this->src); ?>") no-repeat center center fixed;
  -webkit-background-size: cover;
  -moz-background-size: cover;
  -o-background-size: cover;
  background-size: cover;
}

Damit kann man aber kein background-color und andere background-Eigenschaften für HTML mehr aufsetzen. Könntest du dies wie folgt abändern? Die Präfixe sind IMHO mittlerweile auch überflüssig.

html {
  background-image: url("<?= specialchars($this->src) ?>");
  background-repeat: no-repeat;
  background-position: center;
  background-attachment: fixed;
  background-size: cover;
}

Danke.

aschempp commented 5 years ago

Funktioniert es vielleicht wenn du background-color mit !important setzt? 😇

Aybee commented 5 years ago

Ja, das ginge, ich benutze !important aber sehr ungerne, da dies Probleme beim weiteren Überschreiben bringen könnte. Was gefällt dir denn an obiger Lösung nicht?

aschempp commented 5 years ago

Was gefällt dir denn an obiger Lösung nicht?

Nichts besonderes, ich wollte nur keine neue Version veröffentlichen nur um die Zeilen zu ändern 😉 Aber ich lasse das Ticket sicher offen damit es bei den nächsten Anpassungen mit übernommen werden kann.

Aybee commented 5 years ago

Ah, gut, danke.

Aybee commented 5 years ago

Vielen Dank 👍