Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[StyleSheet generator] support data: URIs #3661

Closed
wants to merge 2 commits into from
Closed

[StyleSheet generator] support data: URIs #3661

wants to merge 2 commits into from

Conversation

fbender
Copy link

@fbender fbender commented Nov 30, 2011

Currently, the StyleSheet generator adds ../ to any url() path without a leading /. This also affects data: URIs, which renders them unusable. Thus, I added some more checks.

Additionally, I fixed character encoding for url() values: Previously, it was impossible to use e. g. images.php?id=324e2c&width=32 as a background image, or a Widows-style path. This bug also affected some data: URIs.

Florian Bender added 2 commits November 30, 2011 23:31
@fbender
Copy link
Author

fbender commented Dec 1, 2011

Please fix this in 2.10.4.

@leofeyer
Copy link
Member

leofeyer commented Jan 9, 2012

Fixed in 4580aec.

You can use data: URIs in Contao 2.10 by adding them to the "Custom CSS code" section and wrapping them in single quotes (') instead of double quotes (").

@leofeyer leofeyer closed this Jan 9, 2012
@fbender
Copy link
Author

fbender commented Jan 9, 2012

Thank you!

What about the second commit, i. e. fixing character encoding? Those characters should really be URL-encoded (or literals, at least the equality sign =).

@leofeyer
Copy link
Member

leofeyer commented Jan 9, 2012

I didn't know what that was for. Are there missing decodeEntities=>true in the DCA?

@fbender
Copy link
Author

fbender commented Jan 9, 2012

2nd paragraph in the initial post ;-).

Are there missing decodeEntities=>true in the DCA?

Probably ;-)

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

Successfully merging this pull request may close these issues.

2 participants