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.

Sign up for GitHub

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

Jump to bottom

Covnerter: Handle "currentColor" #23

Open
vancluever opened this issue Dec 26, 2023 · 0 comments
Open

Covnerter: Handle "currentColor" #23

vancluever opened this issue Dec 26, 2023 · 0 comments

Comments

@vancluever
Copy link
Contributor

Observed this being used in some SVGs saved in Inkscape, might not be something that's actually coming from there specifically but possibly some other conversion (e.g. FontForge). Currently, it's unhandled, which sets the image to use magenta, which can hide some other things like alpha issues when checking against images.
vancluever added a commit to vancluever/tvg-powerline that referenced this issue Dec 26, 2023
@vancluever
Update images
d8384de 
This updates the images with the fix in TinyVG/sdk#21.

We also remove the "currentColor" fill attribute for each path, which
allows for a proper defaulting to black versus the placeholder color,
allowing for rendering bleed issues to better be seen. (See
TinyVG/sdk#23).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vancluever