basmilius / weather-icons

Free to use animated weather icons.
https://meteocons.com
MIT License
884 stars 121 forks source link

Partly-cloudy line icons use open cloud #30

Open marcelstoer opened 1 year ago

marcelstoer commented 1 year ago

The partly-cloudy line icons use a cloud symbol that is not closed at the bottom i.e. like those for the rain/hail/etc. clouds.

Examples:

The ones displayed at https://basmilius.github.io/weather-icons/index-line.html, however, don't have this bug.

DadiBit commented 1 year ago

I had noticed the same issue in the past, and I believe this is due to the fact that the node script reduces the svg size by removing also some closing tags, which, I believe, should work fine on browser, but on android (for example) it just breaks the last element.

basmilius commented 5 days ago

Hi, sorry for the late answer. Do you mean that the first column in the screenshot should have a closed cloud? Currently working on v3 and I took over the open variant of v2, but I'm open to change it.

image
marcelstoer commented 4 days ago

Do you mean that the first column in the screenshot should have a closed cloud?

Yes, I think that's what I meant. They look really odd to me. I feel that unless there's precipitation the cloud icon should be closed.

basmilius commented 3 days ago

@marcelstoer Thanks for the clarification. I agree :)