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

Request: Add CDN option for currentColor #8573

Closed
pinoceniccola opened this issue Apr 26, 2023 · 3 comments
Closed

Request: Add CDN option for currentColor #8573

pinoceniccola opened this issue Apr 26, 2023 · 3 comments

Comments

@pinoceniccola
Copy link

I see there is already a closed issue about implementing fill:currentColor, but it was around making it the default option.

Still, it would be nice to add it as a CDN option, like any other color, e.g.:

https://cdn.simpleicons.org/simpleicons/currentcolor

In this way users may benefit from the feature without breaking changes or unintended side-effects.

Thank you for your consideration.

@LitoMore
Copy link
Member

LitoMore commented Apr 26, 2023

I just wanted to let you know that won't work.

You are setting currentColor to an external resource. CSS does not apply cross-document and you've two documents.

@LitoMore
Copy link
Member

LitoMore commented Apr 26, 2023

You can create a div with currentcolor in this way: https://jsfiddle.net/LitoMore/xhfw1jqc/26/

Note:
Please use our JSDelivr CDN (CORS friendly) instead of the cdn.simpleicons.org.

@pinoceniccola
Copy link
Author

ah yes, sorry, should have had some coffe before opening the issue. Thanks for the clarifications anyway.

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

No branches or pull requests

2 participants