search

webalys-hq / streamlinehq-npm

The deprecated Streamline NPM package, which will be deleted on 1st November 2022.
https://medium.com/streamline-icons/how-streamline-narrows-down-its-focus-eac6fdb5c6f2
24 stars 5 forks source link

Building1 icon does not allow a fill color #26

Closed pixel-glyph closed 3 years ago

pixel-glyph commented 3 years ago

When importing the Building1 icon and setting a fill color, it does not get applied due to the structure of the icon, i.e. the shape that forms the front of the building is not a single entity, and so a fill cannot be applied. It would be great if the SVG could be rebuilt to allow a defined fill and prevent any background color from coming through.

Example of what is desired (with a white fill): image

Example of current state (without a fill): image

Specific icon import: import Building1 from '@streamlinehq/streamlinehq/img/streamline-regular/building-1-6nlwop.svg'

Apologies if this is not the right place to raise this issue.

cbrwizard commented 3 years ago

Thank you for reporting this. This issue has been sent to our SVG team

cbrwizard commented 3 years ago

We have replaces the SVG for this image. Please validate it by reinstalling the npm package and checking it out again

pixel-glyph commented 3 years ago

@cbrwizard Awesome, thanks for the quick response!

Unfortunately, after reinstalling the package, the icon appears to be the same.

cbrwizard commented 3 years ago

Looks like you're right, we're having a bug with cache. Could you try downloading an SVG directly from this page meanwhile? https://app.streamlinehq.com/icons/streamline-regular/building-construction/buildings/building-1

pixel-glyph commented 3 years ago

Thanks @cbrwizard. For now I'll use the SVG directly and update once the caching issue has been resolved 👍

cbrwizard commented 3 years ago

We have recently recreated all assets and released a new NPM 3.0.1 version, please install it, fetch newest images and check if it solves your issue.

cbrwizard commented 3 years ago

Closing this issue since I believe it's fixed. Feel free to respond if you think otherwise