Skip to content
This repository has been archived by the owner on Apr 18, 2020. It is now read-only.

added the possibility of customizing the color of different parts #8

Closed
wants to merge 17 commits into from

Conversation

maxdow
Copy link
Contributor

@maxdow maxdow commented Jul 12, 2014

I added two options : "monochrome" a boolean which is false by default and "colors", an object to configure the colors of different elements like sun, cloud etc..

When monochrome is set to false , icons are colored like this
skyiconspreview

You can pass custom colors in the options. I also edited the README to show how to configure this new feature

@bojanvidanovic
Copy link

Awsome! Thanks.

@maxdow
Copy link
Contributor Author

maxdow commented Sep 11, 2014

thanks @B0jan :)

@futhr
Copy link

futhr commented Dec 11, 2014

+1

1 similar comment
@SergioCrisostomo
Copy link

+1

@ghost
Copy link

ghost commented Nov 11, 2015

This is really cute, nice work! I don't plan to merge this, but I'd like to keep this open as an issue to figure out how to address it for #11. I think that CSS can reference SVG elements, so maybe this is as simple as applying a class to the relevant SVG parts so that one can recolor them as desired.

galdiuz and others added 4 commits January 27, 2016 15:52
SLEET was renamed to RAIN_SNOW.
The original SLEET was restored.
The original index.html was also restored.
Added separate coloring for the fogbank.
@Julix91
Copy link

Julix91 commented Jul 4, 2017

What am I missing - why is this not being merged?

@maxdow
Copy link
Contributor Author

maxdow commented Jul 4, 2017

@Julix91 because they want to work with plain SVG. I also think it's a lot better than a canvas ;)

@ghost
Copy link

ghost commented Apr 17, 2020

Skycons is no longer maintained; please fork to do any further work on it!

@ghost ghost closed this Apr 17, 2020
This pull request was closed.
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.

None yet

9 participants