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

feat: Add Tip Component #1048

Closed
wants to merge 18 commits into from
Closed

feat: Add Tip Component #1048

wants to merge 18 commits into from

Conversation

dlnr
Copy link
Contributor

@dlnr dlnr commented Jan 18, 2024

Adding the Badge to the tip raises a couple of questions. Do we need to inherit the font-size of the other child? And do we need to add a required "badge" prop to enforce the badge (and badge color)?

@github-actions github-actions bot temporarily deployed to demo-DES-544-Tip January 19, 2024 14:28 Destroyed
@github-actions github-actions bot temporarily deployed to demo-DES-544-Tip January 19, 2024 14:36 Destroyed
@dlnr dlnr marked this pull request as ready for review January 19, 2024 14:38
@dlnr dlnr requested a review from VincentSmedinga January 19, 2024 14:38
@github-actions github-actions bot temporarily deployed to demo-DES-544-Tip January 23, 2024 13:28 Destroyed
@github-actions github-actions bot temporarily deployed to demo-DES-544-Tip January 23, 2024 13:41 Destroyed
Base automatically changed from feature/DES-331-Badge to develop January 26, 2024 12:30
@github-actions github-actions bot temporarily deployed to demo-DES-544-Tip January 26, 2024 12:43 Destroyed
@VincentSmedinga
Copy link
Contributor

We decided against including this component now. It’s unclear if and how it will be used, and the implementation is not much more than a flex row with a gap and some padding. Until we offer something specific, people can wrap a Badge and a Paragraph in a small custom container.

@VincentSmedinga VincentSmedinga deleted the feature/DES-544-Tip branch January 26, 2024 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants