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

#382 Handle only numeric values as Badge content. Support custom limit. #383

Merged
merged 2 commits into from
Aug 25, 2022

Conversation

sgraczyk
Copy link

@sgraczyk sgraczyk commented Aug 8, 2022

Resolves: #382

Description

  • Badge has two new props: count and max with defaults set to 0 and 99.
  • children prop is no longer supported as only numeric values are allowed.
  • React.HTMLAttributes<HTMLSpanElement> are now supported as additional props.

Storybook

Checklist

Obligatory:

  • Self-review
  • Unit & integration tests
  • Storybook cases
  • Design review
  • Functional (QA) review

Optional:

  • Accessibility cases (keyboard control, correct HTML markup, etc.)

@sgraczyk sgraczyk linked an issue Aug 8, 2022 that may be closed by this pull request
@sgraczyk sgraczyk merged commit 04bc820 into v1 Aug 25, 2022
@sgraczyk sgraczyk deleted the feature/382 branch August 25, 2022 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Badge] Content type improvements
4 participants