changed the variant of the button to show a filter is active

[shramanpaul]

Proposed Changes

• Fixes Log Update - filter #6931

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

Merge Checklist

filterupdate.mp4

@rithviknishad please have a review

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
care-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 22, 2024 5:45pm

[netlify]

✅ Deploy Preview for care-egov-staging ready!

Name	Link
🔨 Latest commit	f6ab080
🔍 Latest deploy log	https://app.netlify.com/sites/care-egov-staging/deploys/65aea99a715e5100086804a2
😎 Deploy Preview	https://deploy-preview-7026--care-egov-staging.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[rithviknishad]

Nice attempt, but I don't think this looks good and also layout shift happens. The filter button becomes smaller when the filter is applied.

Instead, maybe we could change the Button's variant to "primary" when a filter is present and keep it "secondary" if not.

[shramanpaul]

Thanks for the review. I will be making the requested changes as soon as possible

[shramanpaul]

filterupdate02.mp4

made the requested changes please have a review @rithviknishad

[rithviknishad]

Cleaner! LGTM

[nihal467]

LGTM

[github-actions]

@shramanpaul We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌.