Skip to content
This repository has been archived by the owner on Jan 22, 2024. It is now read-only.

Implement sass and compass support #2

Closed
may17 opened this issue May 26, 2014 · 18 comments
Closed

Implement sass and compass support #2

may17 opened this issue May 26, 2014 · 18 comments

Comments

@may17
Copy link
Contributor

may17 commented May 26, 2014

Contao 3.3 ships with sass and compass support. This feature should be represented in the contao demo.

@may17
Copy link
Contributor Author

may17 commented Feb 9, 2015

We decided to drop the Stylesheet Database Support with contao 3.5. So Scss files will be the future of the contao demo.

@MoniqueHahnefeld
Copy link

Hey @may17 Joe.
ich habe jetzt angefangen ein SCSS-bundle zu erarbeiten und werde bis Ende nächster Woche damit fertig sein. Habt ihr auch schon begonnen. Vielleicht kann man sich da absprechen?

Ich habe dafür diese Struktur verwendet. Die scheint mir sehr sinnvoll zu sein.
https://github.com/HugoGiraudel/sass-boilerplate/tree/master/stylesheets

@sascha-mueller
Copy link
Contributor

Hallo Monique,
ich habe schon etwas erstellt und bin damit recht weit fortgeschritten.
Wenn wir also unnötige Arbeit vermeiden wollen, sollten wir uns
unbedingt absprechen.

Gruß,
Sascha

Am 10.07.2015 um 18:07 schrieb Monique Hahnefeld:

Hey @may17 https://github.com/may17 Joe.
ich habe jetzt angefangen ein SCSS-bundle zu erarbeiten und werde bis
Ende nächster Woche damit fertig sein. Habt ihr auch schon begonnen.
Vielleicht kann man sich da absprechen?


Reply to this email directly or view it on GitHub
#2 (comment).

@MoniqueHahnefeld
Copy link

@samuelll Ok, kann ich dich morgen Nachmittag oder Montag :D dazu mal anrufen?

@MoniqueHahnefeld
Copy link

Sascha und Ich haben eben unsere beiden Varianten besprochen und werden beide zur Verfügung stellen und nächste Woche diskutieren wie der Aufbau der Variablen und der Ordner-Struktur sein sollte.

Eure Meinung ist herzlich willkommen @may17 und @allother :)

@samuelll hier siehst du wie Bootstrap das macht:
https://github.com/twbs/bootstrap-sass/blob/master/assets/stylesheets/bootstrap/_variables.scss#L397
Die gehen auch den Weg über detaillierte Benennung nach Komponenten.

@may17
Copy link
Contributor Author

may17 commented Jul 11, 2015

Klar können wir gerne besprechen. Bitte vergesst aber nicht, dass die Demo so flach wie möglich gehalten werden soll. Ich finde Ansätze wie von dir @MoniqueHahnefeld sehr gut aber das Ziel der Demo ist es einfach zu sein ausgehend vom eher unerfahrenen Anwender.

Eine Konvertierung der jetzigen Demo Css samt der Variablen ist für den ersten Schritt vollkommen ausreichend. Zu dem wäre eine Sass Grid implementierung für Contao generell wichtig ebenso wie ein prozentuales Grid.

@MoniqueHahnefeld
Copy link

@samuelll Du wolltest deine Version, was du schon hast auch hochladen. Wann wirst du das tun :) ?
Ich würde gerne einen Konsens finden worauf wir da aufbauen können, was die Struktur angeht.

@MoniqueHahnefeld
Copy link

@may17
"Eine Konvertierung der jetzigen Demo Css samt der Variablen ist für den ersten Schritt vollkommen ausreichend. Zu dem wäre eine Sass Grid implementierung für Contao generell wichtig ebenso wie ein prozentuales Grid." -> warum nicht mehr, wenn es schon gemacht ist? Ich sehe hier den Sinn nicht.
Würdest du mit einer unvollständig durchdachten SCSS-Variante arbeiten wollen?

@sascha-mueller
Copy link
Contributor

Die Files sind schon seit 14. fertig.
Scheint als hab ich das mit github noch nicht so im griff :-/

Am 20.07.2015 um 14:09 schrieb Monique Hahnefeld:

@samuelll https://github.com/Samuelll Du wolltest deine Version, was
du schon hast auch hochladen. Wann wirst du das tun :) ?
Ich würde gerne einen Konsens finden worauf wir da aufbauen können, was
die Struktur angeht.


Reply to this email directly or view it on GitHub
#2 (comment).

@MoniqueHahnefeld
Copy link

@samuelll kannst du nicht einfach ein PR machen wie ich auch oder ein Fork und den Hier verlinken?

@may17
Copy link
Contributor Author

may17 commented Jul 30, 2015

@MoniqueHahnefeld es geht bei der COD nicht darum einen Startpunkt für das erste eigene Projekt zu haben. Die Demo ist zum testen und rumspielen da. Etwas anderes gibt die Lizenz gar nicht her da shareAlike ;) Daher "keep it simple and stupid".

@MoniqueHahnefeld
Copy link

@may17 Also @samuelll war auch der Meinung das eine SCSS-Variante angebracht ist. Ich versteh deinen Ansatz ehrlich gesagt überhaupt nicht. @leofeyer Hat die Letzte Music Academy auch als SCSS-Variante gebracht. Und damit kann man auch besser herumspielen und lernen.

@sascha-mueller
Copy link
Contributor

@MoniqueHahnefeld Eine SCSS-Variante stand auch nie in Frage.
Lediglich bei der Tiefe der Ausgestaltung seit Ihr nicht einer Meinung.

Ich bin mir selbst noch nicht sicher wie tief diese gehen soll, kann mir aber durchaus vorstellen, dass einige nicht so erfahrene Interessenten auf der Strecke bleiben, wenn man alle SCSS-Register zieht.

@MoniqueHahnefeld
Copy link

@samuell Ja ich habe das ja verstanden das meine Variante vielleicht zu komplex für einen Anfänger ist. Ich finde es aber wenig konstruktiv wenn man einfach pauschal sagt: "Das nehmen wir so nicht"
Wir können doch hier eine sinnvolle Struktur diskutieren und uns darauf dann festlegen. Ein SCSS-Package nützt finde ich nix wenn die Struktur nicht gut ist.

Ich wollte gerne was mit meinen SCSS Erfahrungen zum Projekt beitragen. BTW: Hast du deine Variante denn jetzt hochgeladen bekommen?

@sascha-mueller
Copy link
Contributor

@MoniqueHahnefeld Ohne jetzt genau nachzulesen hab ich nicht "Das nehmen wir so nicht" verstanden. Eher habe ich was von "sehr guten Ansätzen" in Erinnerung.

Eine sinnvolle Struktur ist Projekt- und Geschmacksabhängig.
Ich habe aktuell gerade ein spannendes Projekt wo ich wieder eine andere Ordner - und Dateistruktur vorfinde, als ich sie bisher gesehen oder genutzt habe. Gute Struktur definiert einfach jeder anders.

Für die COD habe ich versucht Sie ziemlich einfach zu halten.
Das Ergebnis findest Du hier:
https://github.com/contao/official-demo/tree/master/src/files/contaodemo/scss

Was ich als nächste größere Schritte für sinnvoll erachte ist der Austausch des Grids, wie es auch bereits in einem der Tickets angedacht wurde.
Danach wäre es aus meiner Sicht sinnig das CSS einem Rewrite zu unterziehen und damit gern auch die Datei- und Ordnerstruktur zu überarbeiten.

Deine Unterstützung ist uns weiterhin willkommen.

@may17
Copy link
Contributor Author

may17 commented Jul 31, 2015

@MoniqueHahnefeld ich habe in keinem Satz gesagt "Das nehmen wir so nicht". Das ist schlicht eine Unterstellung.

Ich verstehe noch nicht genau was deine Mission gerade ist? Ich habe eher das Gefühl, dass es dir darum geht die COD so zu verändern wie es dir passt. So funktioniert das aber nicht. Wenn du dich in die COD einbringen willst wäre es sinnvoll das Gespräch mit den beteiligten zu Suchen anstelle hier wilde Diskussionen anzufangen oder dein eigenes Ding durchzuziehen.

@MoniqueHahnefeld
Copy link

@may17 Nein das verstehst du falsch. Ich warte nur seit dem PR mal auf ein konkretes Feedback. Sowas wie: Die Variablen übernehmen wir. Oder die Ordner sind auf jeden Fall sinnvoll. Einfach damit es weiter geht. Entschuldige wenn ich da etwas ungeduldig bin. Und ich wüsste jetzt auch nicht warum wir das nicht hier ausdiskutieren können. Dafür sind doch die Tickets und es kann so jeder seine Meinung dazu geben, oder etwa nicht?

@may17
Copy link
Contributor Author

may17 commented Aug 2, 2015

Ich wüsste nicht über was es zu diskutieren gibt. Dieses Ticket hat nichts mit deinem Pull request zu tun. Du musst dich wohl oder übel gedulden.

@may17 may17 closed this as completed Aug 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants