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

allow eval expressions for icon-image and fill-image #56

Merged
merged 1 commit into from
Aug 13, 2016

Conversation

DerDakon
Copy link
Contributor

@DerDakon DerDakon commented Oct 5, 2015

If those tags contained an eval expression those would erroneously be taken as a filename, leading to a parse error. Simply check if the expression is an eval before calling the filename handling.

This does not make it really work, but it at least does not crash anymore.

This does not fix #28, but is the first step in that direction.

@DerDakon DerDakon force-pushed the icon-eval branch 3 times, most recently from 4f79ec5 to c249f6b Compare November 21, 2015 21:00
@DerDakon DerDakon force-pushed the icon-eval branch 2 times, most recently from 5a0223f to 0286ec3 Compare December 11, 2015 18:40
@DerDakon DerDakon force-pushed the icon-eval branch 2 times, most recently from c5168bb to 9a7437c Compare January 5, 2016 22:02
@DerDakon DerDakon force-pushed the icon-eval branch 2 times, most recently from 31b5c6b to 61b79c8 Compare January 26, 2016 21:07
@DerDakon DerDakon force-pushed the icon-eval branch 2 times, most recently from 7174f0c to dd12874 Compare March 14, 2016 06:50
If those tags contained an eval expression those would erroneously be taken as
a filename, leading to a parse error. Simply check if the expression is an eval
before calling the filename handling.

This does not make it really work, but it at least does not crash anymore.
@rurseekatze rurseekatze merged commit 8f134c0 into rurseekatze:master Aug 13, 2016
@DerDakon DerDakon deleted the icon-eval branch August 13, 2016 11:29
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.

icon-image properties that are dynamically created are not supported
2 participants