-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Sistent Card Component #6117
base: master
Are you sure you want to change the base?
Sistent Card Component #6117
Conversation
🚀 Preview for commit 33df114 at: https://6756a2a4e25791afc5b16274--layer5.netlify.app |
33df114
to
ade40a3
Compare
🚀 Preview for commit ade40a3 at: https://6756ad5a8a59a4c1a351345a--layer5.netlify.app |
Thank you, @Anand-Theertha 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Anand-Theertha I have reviewed your PR I have put some comments; you can look into it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import { useStyledDarkMode } from "../../../../../theme/app/useStyledDarkMode"; | ||
|
||
const codes = [ | ||
`const cardOutlined = ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than creating a new component, you can directly pass as a child component to Card. So that I will be easier to copy for developers in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made it this way to keep the code modular and abstract to make it more reusable. Do you still suggest I add this as a child component? @Vidit-Kushwaha
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same increase in contrast between the text and background.
Will make these changes, thanks @Vidit-Kushwaha |
Hi @Anand-Theertha Facing issues merging your PR in Sistent? We are conducting a special tutorial on this Monday, December 16, 2024, at 7 AM Central (6:30 PM IST) in the website meeting. We'll cover:
Get more information: here 👥 Get involved:
Don’t miss out. 🚀 |
@Anand-Theertha did you made changes based on feedback yet? |
@Anand-Theertha checking in again? |
Will push these changes today, thanks for the reminder @sudhanshutech |
? |
Hi @Anand-Theertha, would appreciate if you could implement the reviews and push the changes , so that we can merge this? |
Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Vrushabh <vrushabhgawas14@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Remove 404 link, and swap for 'Appears on publication' content Fixes layer5io#5885 Signed-off-by: GitHub <noreply@github.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Remove unused import Button Signed-off-by: GitHub <noreply@github.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
….mdx Signed-off-by: Lee Calcote <leecalcote@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
… page Signed-off-by: Lee Calcote <lee.calcote@layer5.io> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
…rn themes Signed-off-by: Lee Calcote <lee.calcote@layer5.io> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Lee Calcote <lee.calcote@layer5.io> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Lee Calcote <lee.calcote@layer5.io> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Ajay Singh <37625112+Ajay-singh1@users.noreply.github.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Ajay Singh <37625112+Ajay-singh1@users.noreply.github.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: l5io <ci@layer5.io> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: ritik-malik <ritik18406@iitd.ac.in> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: ritik-malik <ritik18406@iitd.ac.in> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Co-authored-by: Vidit Kushwaha <viditkushwaha530@gmail.com> Signed-off-by: maalik <ritik18406@iiitd.ac.in> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Bumps [eslint-plugin-react](https://github.com/jsx-eslint/eslint-plugin-react) from 7.36.1 to 7.37.3. - [Release notes](https://github.com/jsx-eslint/eslint-plugin-react/releases) - [Changelog](https://github.com/jsx-eslint/eslint-plugin-react/blob/master/CHANGELOG.md) - [Commits](jsx-eslint/eslint-plugin-react@v7.36.1...v7.37.3) --- updated-dependencies: - dependency-name: eslint-plugin-react dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Bumps [@emotion/styled](https://github.com/emotion-js/emotion) from 11.13.0 to 11.14.0. - [Release notes](https://github.com/emotion-js/emotion/releases) - [Changelog](https://github.com/emotion-js/emotion/blob/main/CHANGELOG.md) - [Commits](https://github.com/emotion-js/emotion/compare/@emotion/styled@11.13.0...@emotion/styled@11.14.0) --- updated-dependencies: - dependency-name: "@emotion/styled" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Bumps [swiper](https://github.com/nolimits4web/Swiper) from 10.3.1 to 11.1.15. - [Release notes](https://github.com/nolimits4web/Swiper/releases) - [Changelog](https://github.com/nolimits4web/swiper/blob/master/CHANGELOG.md) - [Commits](nolimits4web/swiper@v10.3.1...v11.1.15) --- updated-dependencies: - dependency-name: swiper dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: TheFaheem <faheemmushtaq89@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
Signed-off-by: Shivam Gupta <sudhanshudasgupta9@gmail.com> Signed-off-by: Anand-Theertha <anandtheerthav3@gmail.com>
ad5e378
to
4d0db2c
Compare
🚀 Preview for commit 4d0db2c at: https://677ab9604f3ddfa7b38228fb--layer5.netlify.app |
I pushed my recent changes, but when I tried to follow the steps for DCO a lot more commits from the master branch were pushed. Can someone please guide me as to what can be done to push just my changes? |
Description
This PR fixes #5908. This PR adds a Card component to the Sistent components. Following are a few screenshots of how the implementation looks:
Notes for Reviewers
Signed commits