Sistent Card Component #6117
Sistent Card Component #6117
Conversation
|
🚀 Preview for commit 33df114 at: https://6756a2a4e25791afc5b16274--layer5.netlify.app |
Signed-off-by: Anand-Theertha <[email protected]>
Signed-off-by: Anand-Theertha <[email protected]>
Signed-off-by: Anand-Theertha <[email protected]>
Signed-off-by: Anand-Theertha <[email protected]>
Anand-Theertha
force-pushed
the
sistent-card
branch
from
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.
@Anand-Theertha I have reviewed your PR I have put some comments; you can look into it
There was a problem hiding this comment.
Text on the card is not properly visible Can you use other shades?
| 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.
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 |
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