Skip to content

Conversation

M-DEV-1
Copy link
Member

@M-DEV-1 M-DEV-1 commented Apr 9, 2025

Description

This PR fixes #5960

I've added the Switch component to the Sistent Components page with code examples, usage and guidance.

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Apr 9, 2025

🚀 Preview for commit 3a4450c at: https://67f68128b244c6f132e32a86--layer5.netlify.app

@vr-varad
Copy link
Contributor

vr-varad commented Apr 9, 2025

  1. Could Increase Visibility
    image
  2. Missing Tabs
    image
  3. Could Be aligned
    image

@vr-varad
Copy link
Contributor

vr-varad commented Apr 9, 2025

Hey @M-DEV-1 some fixes

@M-DEV-1
Copy link
Member Author

M-DEV-1 commented Apr 9, 2025

Thanks @vr-varad for pointing these out, I'll fix them 🤝

I'm not sure what you meant by 3., but I will be adding more switches in the same code block for better representation.

@vr-varad
Copy link
Contributor

vr-varad commented Apr 9, 2025

Thanks @vr-varad for pointing these out, I'll fix them 🤝

I'm not sure what you meant by 3., but I will be adding more switches in the same code block for better representation.

I meann't I would look good if they were side by side but nevermind

@M-DEV-1
Copy link
Member Author

M-DEV-1 commented Apr 9, 2025

Thanks @vr-varad for pointing these out, I'll fix them 🤝
I'm not sure what you meant by 3., but I will be adding more switches in the same code block for better representation.

I meann't I would look good if they were side by side but nevermind

Yes, you're right. It would be better side by side, in the same block. I'll make those changes. Thanks.

@vishalvivekm
Copy link
Contributor

@M-DEV-1
Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST

Add it as an agenda item to the meeting minutes, if you would :)

@vishalvivekm
Copy link
Contributor

Thank you for @vr-varad offering feedback here, 👏

@l5io
Copy link
Contributor

l5io commented Apr 14, 2025

🚀 Preview for commit 8f0d5d4 at: https://67fcc9f59f81bdc6026cd179--layer5.netlify.app

@vishalvivekm
Copy link
Contributor

@M-DEV-1 are all the suggestions in here by @vr-varad and during last sites' call, incorporated?

@l5io
Copy link
Contributor

l5io commented Apr 21, 2025

🚀 Preview for commit ff3b43d at: https://680627b848abe9f284a11437--layer5.netlify.app

@vishalvivekm
Copy link
Contributor

Thank you @Souradip121 for making these changes, however, a couple of them are still in need of being included in here.

@sudhanshutech
Copy link
Member

sudhanshutech commented Apr 29, 2025

it looks good to me @vishalvivekm @amitamrutiya

Copy link

stale bot commented Jun 27, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the issue/stale Issue has not had any activity for an extended period of time label Jun 27, 2025
@stale stale bot removed the issue/stale Issue has not had any activity for an extended period of time label Jun 28, 2025
@vishalvivekm
Copy link
Contributor

@M-DEV-1 failing build / checks

Copy link
Contributor

@Ayushmore1214 Ayushmore1214 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
I think this needs some formatting and also side by side will looks great

Copy link

stale bot commented Jul 18, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the issue/stale Issue has not had any activity for an extended period of time label Jul 18, 2025
@vr-varad vr-varad closed this Jul 20, 2025
@vr-varad vr-varad reopened this Jul 20, 2025
@stale stale bot removed the issue/stale Issue has not had any activity for an extended period of time label Jul 20, 2025
@vr-varad
Copy link
Contributor

Any updates @M-DEV-1

Copy link

stale bot commented Jul 29, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the issue/stale Issue has not had any activity for an extended period of time label Jul 29, 2025
@vr-varad vr-varad closed this Aug 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue/stale Issue has not had any activity for an extended period of time project/sistent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Sistent] Add Switch component to the sistent components page
6 participants