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

Render icons from SVG #7483

Closed
wants to merge 8 commits into from
Closed

Render icons from SVG #7483

wants to merge 8 commits into from

Conversation

@luke-jr
Copy link
Member

luke-jr commented Feb 8, 2016

This probably doesn't make sense to merge to Core itself due to the added dependencies (rsvg & imagemagick; and icnsutils for Mac) and risks (of installed rsvg mis-rendering the SVG), but posting it here (insta-close) for potential review and/or cherry-picking.

@luke-jr luke-jr closed this Feb 8, 2016
@luke-jr luke-jr reopened this Feb 8, 2016
@luke-jr luke-jr force-pushed the luke-jr:svg_icon branch Feb 8, 2016
@luke-jr luke-jr closed this Feb 8, 2016
@luke-jr luke-jr reopened this Feb 8, 2016
@luke-jr luke-jr force-pushed the luke-jr:svg_icon branch Feb 8, 2016
@luke-jr
Copy link
Member Author

luke-jr commented Feb 8, 2016

(Leaving it open for final branch updates/Travis testing)

@luke-jr luke-jr force-pushed the luke-jr:svg_icon branch 5 times, most recently Feb 8, 2016
@luke-jr luke-jr force-pushed the luke-jr:svg_icon branch 4 times, most recently Feb 9, 2016
@luke-jr luke-jr force-pushed the luke-jr:svg_icon branch to da30a01 Feb 9, 2016
@luke-jr
Copy link
Member Author

luke-jr commented Feb 10, 2016

Ok, this seems to be working good now.

@luke-jr luke-jr closed this Feb 10, 2016
luke-jr added a commit to bitcoinknots/bitcoin that referenced this pull request Jun 30, 2016
@rebroad
Copy link
Contributor

rebroad commented Sep 25, 2016

Why the instaclose? If it's good enough for Knots, then why wouldn't it be good enough for Core? And if it's not needed for Core, then why is it needed for Knots?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.