Skip to content

[POC] Inline small svg files in css#6020

Closed
rullzer wants to merge 1 commit into
masterfrom
scss_inline_svg
Closed

[POC] Inline small svg files in css#6020
rullzer wants to merge 1 commit into
masterfrom
scss_inline_svg

Conversation

@rullzer

@rullzer rullzer commented Aug 8, 2017

Copy link
Copy Markdown
Member

A simple helper function for the SCSS compiler to include svg files.

A simple helper function for the SCSS compiler to include svg files.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer added this to the Nextcloud 13 milestone Aug 8, 2017
@MorrisJobke

Copy link
Copy Markdown
Member

So this checks all existing url() sections in SCSS and replaces them with inline stuff if it is an SVG? How about relative paths? How about stuff inside of apps that refer to the server images?

@rullzer

rullzer commented Aug 8, 2017

Copy link
Copy Markdown
Member Author

Yes that will fail for now... then we have to do more url magic etc.

@MorrisJobke MorrisJobke mentioned this pull request Aug 8, 2017
@pixelipo

pixelipo commented Aug 8, 2017

Copy link
Copy Markdown
Contributor

👎 see my comment in #5985

@MorrisJobke MorrisJobke closed this Aug 8, 2017
@MorrisJobke MorrisJobke deleted the scss_inline_svg branch August 8, 2017 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants