search

FriendsOfREDAXO / minify

Minifiziert und bündelt CSS/Sass, JavaScript, HTML
https://github.com/FriendsOfREDAXO/minify
MIT License
43 stars 5 forks source link

Nur Pfad zur CSS/JS-Datei #5

Closed alxndr-w closed 8 years ago

alxndr-w commented 8 years ago

Ich benötige den Pfad zur CSS ohne -Tag. Gibt's eine einfache Möglichkeit, da ranzukommen?

phoebusryan commented 8 years ago

Nein, aktuell nicht. Das wird ja direkt eingefügt. Was hast du vor?

alxndr-w commented 8 years ago

Kritisches CSS inline ausgeben. Ich habe einfach den Code aus der boot.php abgewandelt und für mich angepasst.

phoebusryan commented 8 years ago

Okay, der wird dann aber mit dem nächsten Update überschrieben.

Und was ist kritisches CSS?

alxndr-w commented 8 years ago

https://www.smashingmagazine.com/2015/08/understanding-critical-css/

Ich hab den Code nicht ins Addon, sonst würde er überschrieben, ja. Aber den Code als Vorlage in mein project-Addon.

sebastiannoell commented 8 years ago

Wäre auch an der Lösung interessiert.

phoebusryan commented 8 years ago

Okay, ja grundsätzlich könnte ich eine Methode bauen, die den Namen zurückgibt. Dann kannst du damit machen was du willst.

phoebusryan commented 8 years ago

Also ihr möchtet die Möglichkeit, zusätzlich zu den Platzhaltern ein Set via API-Methode als Code statt File zu bekommen?

alxndr-w commented 8 years ago

Dateiname hätte mir genügt, aber ja, darauf läuft es hinaus :)

vlt. reicht es, das über einen zusätzlichen Parameter lösen, bspw. output=inline und output=link (default)?

phoebusryan commented 8 years ago

... und bei inline würde z.b. ein style-tag statt link-tag hingesetzt werden? das wäre simpel umsetzbar

alxndr-w commented 8 years ago

So würde ich mir das vorstellen. @sebastiannoell ?

sebastiannoell commented 8 years ago

Yes! Klingt super.

phoebusryan commented 8 years ago

Okay, dann werde ich das so machen. Eine Frage zur Umsetzung: Einerseits könnte man den Platzhalter ergänzen (wie alexplusde es vorgeschlagen hat) à la 'output=inline oder output=link' ... oder aber mal legt im Backend den Outputmodus fest. Man möchte ja eigentlich nie das gleiche Set als Inline und als link haben oder?

alxndr-w commented 8 years ago

Selbst wenn man es wollen würde, was eigentlich keinen Sinn ergibt, könnte man das set immer noch doppelt anlegen, um den gleichen Effekt zu bekommen. Ist mir also egal. Gegen eine Einstellung im Backend hätte ich nichts einzuwenden.

staabm commented 8 years ago

wenn der use-case kritisches css ist, dann macht eine einmalige stelle im backend keinen sinn (da man ja nicht-kritisches css dann als datei verlinkt und kritisches inline hat)

phoebusryan commented 8 years ago

Das ist schon richtig, jedoch wären kritisches und nicht-kritisches css ja zwei verschiedene Sets. Von dem her würde es dann passen.