sistent Icons set to filled by default

Current Behavior

Related to https://github.com/layer5io/sistent/issues/654...

🎨 Wireframes and designs for Sistent site in Figma (open invite)

Jun 25 '24 21:06 leecalcote

Hi, I would like to take this issue. Can you assign it to me? I have already cloned the repo, and now I am trying to locate that icon in the code.

Sep 10 '24 18:09 heysujal

@heysujal sounds great!

Sep 11 '24 00:09 leecalcote

I have done the local setup of project and also made changes in sistent component , but I am not able to find where this is being used in Layer5 site

Oct 04 '24 08:10 heysujal

I have tried to make changes in sistent, then build it. After doing this, I installed it in Layer5 and tried to use it. For some reason, it is not able to pickup the changes I have made to add the fill property. These are the changes I have made EditIcon.tsx

404.js in Layer5. (Just for testing out)

Output

Oct 04 '24 16:10 heysujal

@heysujal, apologies, I've not seen you're messages until now. I'm tagging others, who can assist, though. @nebula-aac @sudhanshutech @SAHU-01 @hargunkaur286 @amitamrutiya

Oct 11 '24 16:10 leecalcote

@leecalcote @sudhanshutech Hey is this issue still open :)

Aug 20 '25 18:08 ShivanshCharak

@vr-varad, please triage.

Aug 20 '25 19:08 leecalcote

Hey @ShivanshCharak it is open for everyone. But keep in mind, the issue is about the Icons being filled and themed at the same time.

Aug 21 '25 11:08 vr-varad