Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add toggle-button component #6127

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

eyeaadil
Copy link

@eyeaadil eyeaadil commented Dec 19, 2024

Description

This PR fixes #5958

Notes for Reviewers

For complete changes check the given link
https://6764ee9243e3a0d6fe4821ce--layer5.netlify.app/projects/sistent/components/toggle-button/guidance

ScreenShot
image

image

image

image

image

Signed commits

  • Yes, I signed my commits.

@eyeaadil eyeaadil force-pushed the add-toggle-button-component branch from bb056ec to fe4e389 Compare December 19, 2024 14:53
@l5io
Copy link
Contributor

l5io commented Dec 19, 2024

🚀 Preview for commit fe4e389 at: https://6764364266069e46100ebd59--layer5.netlify.app

@eyeaadil
Copy link
Author

@l5io ,can you please review this PR?

Copy link

@Pratik-050 Pratik-050 left a comment

Choose a reason for hiding this comment

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

Also make sure to include a screenshot or screen recording of the changes you've made, that'll make the review process easier.
Revert the changes in package-lock.json and gatsby-node.js

gatsby-node.js Outdated
@@ -728,6 +728,9 @@ exports.createPages = async ({ actions, graphql, reporter }) => {

const components = componentsData.map((component) => component.src.replace("/", ""));


console.log("Adilllllllllll",components);

Choose a reason for hiding this comment

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

Why are there logs? You should remove all of them.

@l5io
Copy link
Contributor

l5io commented Dec 20, 2024

🚀 Preview for commit 1f86b76 at: https://6764e31e62c055bf7dc6cd61--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Dec 20, 2024

🚀 Preview for commit ee92d60 at: https://6764ee9243e3a0d6fe4821ce--layer5.netlify.app

@eyeaadil
Copy link
Author

@Vidit-Kushwaha ,can you please review this?

@Pratik-050
Copy link

@eyeaadil still I can see changes in package.json and gatsby-node.js

Copy link
Member

@sudhanshutech sudhanshutech left a comment

Choose a reason for hiding this comment

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

@eyeaadil revert changes in package lock file

@sudhanshutech
Copy link
Member

@eyeaadil
Thank you for your contribution!
Let's discuss this during the website call on Monday at 6:30 PM IST (7:00 AM CT).

Please add it as an agenda item to the meeting minutes.

Copy link
Contributor

Choose a reason for hiding this comment

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

Revert changeous in this file

Copy link
Contributor

Choose a reason for hiding this comment

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

Same

Copy link
Contributor

Choose a reason for hiding this comment

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

Button name is not intuitive can we change button name from Bold, Underline, and italic to another name like toogle button 1

@@ -19,7 +19,7 @@ const componentsData = [
id: 3,
name: "Modal",
description:
"A text input is made up of multiple elements that combine to form a component that helps users to read, write, and edit text in an interface.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you revert changes to original for Modal object?

Copy link
Contributor

@Vidit-Kushwaha Vidit-Kushwaha left a comment

Choose a reason for hiding this comment

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

@eyeaadil I have added a few comments. You can look into it

@l5io
Copy link
Contributor

l5io commented Dec 26, 2024

🚀 Preview for commit 787218b at: https://676cd9847770b4210ed2db12--layer5.netlify.app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[Sistent] Add ToggleButton component to the sistent components page
5 participants