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
[PoC] inline svg files #4059
[PoC] inline svg files #4059
Conversation
Saves loading a lot of resources. And since the css file is cached anyway this can save precious seconds. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* @copyright Copyright (c) 2015, Hendrik Leppelsack <hendrik@leppelsack.de> | ||
* @copyright Copyright (c) 2015, Jan-Christoph Borchardt <hey@jancborchardt.net> | ||
* | ||
* @license GNU AGPL version 3 or any later version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we add a message here saying: "This file is automatically generated"?
@@ -0,0 +1,32 @@ | |||
<?php |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also execute this on CI and throw an error if there is a change. 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah was thinking about that as well of course
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some refinements, but looks good code wise :)
This alone removes 13 of 129 requests of the files app (10%) 🙌 |
May be an alternative to #3385 |
We would need a way that the »build process« for this is automated. Also, the icons.scss file has other things than just .svg links. :) |
Lets kill this. We can only do this properly if we would have a compile step or something for it (like scss or javascript). For now not worth it I think... |
Ref #1066 if we want to work on anything like that again. :) |
Saves loading a lot of resources. And since the css file is cached
anyway this can save precious seconds.
Todo:
Files so far:
@ChristophWurst as discussed.