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

Editiermodus: Bilder im Block auf max-width: 100% #739

Closed
alxndr-w opened this issue Jun 15, 2016 · 19 comments
Closed

Editiermodus: Bilder im Block auf max-width: 100% #739

alxndr-w opened this issue Jun 15, 2016 · 19 comments
Labels
Enhancement Improvements for existing features

Comments

@alxndr-w
Copy link
Contributor

Aktuell überläuft der Inhalt.

image

Allgemeine Frage: Gibt es best practices, wie man zusätzliche CSS-Styles im Backend einbindet? Wo findet man da Infos?

@alxndr-w alxndr-w changed the title Editiermodus: Bilder im Block auf max-wdith: 100% Editiermodus: Bilder im Block auf max-width: 100% Jun 15, 2016
@tbaddade
Copy link
Member

Bilder im Block auf max-width: 100%

Für die Ausgabe der Daten ist der Developer verantwortlich. Ich möchte da nichts verzerren.

Gibt es best practices, wie man zusätzliche CSS-Styles im Backend einbindet?

Ich würde ein AddOn schreiben, welches dein CSS in das Backend einbindet.

@gharlan gharlan closed this as completed Jun 20, 2016
@gharlan gharlan added Rejected Enhancement Improvements for existing features labels Jun 20, 2016
@staabm
Copy link
Member

staabm commented Jul 20, 2016

Nochmals gemeldet : http://www.redaxo.org/de/forum/post119629.html#p119629

@staabm
Copy link
Member

staabm commented Jul 20, 2016

Ich pers. Finde dass man schon erwarten könnte dass der content nicht das layout zerstören kann per default

@abra100pro
Copy link

+1 - das macht in keinem Fall einen Sinn. der Developer schreibt ja auch nicht das Backend... vielleicht sehe ich da nicht alles, aber dieser issue ist def. unschön im Moment.
Das einzig Gute: Man sieht sofort, wenn ein Kunde ein 4000x4000px Bild hochlädt ;-)

@tbaddade
Copy link
Member

Man sieht sofort, wenn ein Kunde ein 4000x4000px Bild hochlädt ;-)

Das hat durchaus seine Berechtigung. Was man sieht ist aber, das der Admin versucht, ein Bild von 4000px direkt auf der Website auszuliefern.

@alxndr-w
Copy link
Contributor Author

alxndr-w commented Sep 14, 2016

gilt auch für <video>, btw.

@staabm
Copy link
Member

staabm commented Sep 14, 2016

Das hat durchaus seine Berechtigung. Was man sieht ist aber, das der Admin versucht, ein Bild von 4000px direkt auf der Website auszuliefern.

@tbaddade könnte man das nicht auch via CSS (durch nen overlay element) o.ä. ersichtlich machen dass die bilder eigentlich zu groß sind, aber halt eben nicht das layout auch kaputt machen?

@abra100pro
Copy link

könnte man das nicht auch via CSS (durch nen overlay element) o.ä. ersichtlich machen dass die bilder eigentlich zu groß sind, aber halt eben nicht das layout auch kaputt machen?

Soweit ich das verstehe, sollte man mit dem Media Manager für entsprechende Ausgabe sorgen, das lässt sich ja dann auch im Input (Backend)-Bereich eines Moduls integrieren. Das alleine verhindert natürlich nicht das Hochladen eines 8000x5000px Bildes... Stimmt diese Überlegung?

@staabm
Copy link
Member

staabm commented Sep 14, 2016

Soweit ich das verstehe, sollte man mit dem Media Manager für entsprechende Ausgabe sorgen, das lässt sich ja dann auch im Input (Backend)-Bereich eines Moduls integrieren. Das alleine verhindert natürlich nicht das Hochladen eines 8000x5000px Bildes... Stimmt diese Überlegung?

nicht jeder verwendet für alle bilder überall den media manager. ich würde mit so einer maßnahme nur den "allgemeinen fehlerfall" weniger deutlich "kaputt" aussehen lassen

@gharlan gharlan removed the Rejected label Mar 29, 2017
@gharlan gharlan reopened this Mar 29, 2017
@gharlan
Copy link
Member

gharlan commented Mar 29, 2017

Nach Rücksprache mit @tbaddade mache ich hier wieder auf.
Kann jemand einen PR dafür erstellen?

@skerbis
Copy link
Contributor

skerbis commented Mar 29, 2017

Würde ich gerne machen @gharlan aber es ist mir nicht klar in welche SCSS das rein soll.
iframe, img, video, ,object, embed {max-width: 100%;}

@alxndr-w
Copy link
Contributor Author

iframe? Im Grunde kommen alle Elemente infrage, die von Haus aus eine fixe Größe erzwingen können.

@tbaddade
Copy link
Member

table sehe ich da nicht drin. Es könnten sich dann die einzelnen Zellen überlagern.

@skerbis
Copy link
Contributor

skerbis commented Mar 29, 2017

@tbaddade in welches SCSS muss es denn nun, da sind so viele? .. Oder machst Du?

@tbaddade
Copy link
Member

in welches SCSS muss es denn nun, da sind so viele?

Wenn es für alle Themes gelten soll:
https://github.com/redaxo/redaxo/blob/master/redaxo/src/addons/be_style/scss/_base.scss#L26

Wenn es nur für das REDAXO-Theme gelten soll:
https://github.com/redaxo/redaxo/blob/master/redaxo/src/addons/be_style/plugins/redaxo/scss/_scaffolding.scss#L17

@skerbis
Copy link
Contributor

skerbis commented Mar 30, 2017

@alexplusde
Erledigt
#1181
d0247a3

@staabm staabm closed this as completed Mar 30, 2017
@alxndr-w
Copy link
Contributor Author

Danke! :)

@alxndr-w
Copy link
Contributor Author

alxndr-w commented Apr 4, 2017

@skerbis #1021

@skerbis
Copy link
Contributor

skerbis commented Sep 15, 2017

Das Problem liegt noch in den AddOns vor – Hier in einer README.
bildschirmfoto 2017-09-16 um 01 10 34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvements for existing features
Development

No branches or pull requests

6 participants