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

Nur Pfad zur CSS/JS-Datei #5

Closed
alxndr-w opened this issue Aug 5, 2016 · 15 comments
Closed

Nur Pfad zur CSS/JS-Datei #5

alxndr-w opened this issue Aug 5, 2016 · 15 comments

Comments

@alxndr-w
Copy link
Member

alxndr-w commented Aug 5, 2016

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

@phoebusryan
Copy link
Member

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

@alxndr-w
Copy link
Member Author

alxndr-w commented Aug 8, 2016

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

@phoebusryan
Copy link
Member

phoebusryan commented Aug 8, 2016

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

Und was ist kritisches CSS?

@alxndr-w
Copy link
Member Author

alxndr-w commented Aug 8, 2016

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
Copy link

sebastiannoell commented Aug 8, 2016

Wäre auch an der Lösung interessiert.

@phoebusryan
Copy link
Member

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

@phoebusryan
Copy link
Member

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
Copy link
Member Author

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
Copy link
Member

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

@alxndr-w
Copy link
Member Author

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

@sebastiannoell
Copy link

sebastiannoell commented Aug 10, 2016

Yes! Klingt super.

@phoebusryan
Copy link
Member

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
Copy link
Member Author

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
Copy link
Member

staabm commented Aug 11, 2016

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
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants