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 👍 |
Vidit-Kushwaha
left a comment
There was a problem hiding this comment.
@Anand-Theertha I have reviewed your PR I have put some comments; you can look into it
| import { useStyledDarkMode } from "../../../../../theme/app/useStyledDarkMode"; | ||
|
|
||
| const codes = [ | ||
| `const cardOutlined = ( |
There was a problem hiding this comment.
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.
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.
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: 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? |
you can create a separate new PR with required changes that can be added or you can cheery pick commits |
|
Another PR has been created with only the required 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