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(react-tags): add overflow story #28012

Merged
merged 10 commits into from
Jun 16, 2023

Conversation

YuanboXue-Amber
Copy link
Contributor

@YuanboXue-Amber YuanboXue-Amber commented May 26, 2023

  • Add overflow story for TagGroup:
    image

  • Update Tag root type from NonNullable<Slot<'button'>> to NonNullable<Slot<'button', 'span'>>. When tag is in menuItem, it should be just a visual component

@fabricteam
Copy link
Collaborator

fabricteam commented May 26, 2023

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-tags
Tag
21.277 kB
7.786 kB
21.343 kB
7.822 kB
66 B
36 B

🤖 This report was generated against 89f6b7c0835aca7ce29aff3c9a22ace4eb1b01d5

@size-auditor
Copy link

size-auditor bot commented May 26, 2023

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 89f6b7c0835aca7ce29aff3c9a22ace4eb1b01d5 (build)

@codesandbox-ci
Copy link

codesandbox-ci bot commented May 26, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 48a4b18:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@YuanboXue-Amber YuanboXue-Amber changed the title [draft] tag overflow story feat(react-tags): add overflow story Jun 5, 2023
@YuanboXue-Amber YuanboXue-Amber marked this pull request as ready for review June 5, 2023 16:43
@YuanboXue-Amber YuanboXue-Amber requested review from a team as code owners June 5, 2023 16:43
@YuanboXue-Amber YuanboXue-Amber enabled auto-merge (squash) June 16, 2023 07:47
@YuanboXue-Amber YuanboXue-Amber merged commit 676dc58 into microsoft:master Jun 16, 2023
@YuanboXue-Amber YuanboXue-Amber deleted the tag-overflow branch June 16, 2023 08:26
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 20, 2023
* master: (32 commits)
  applying package updates
  feat(react-tags): add styles for size (microsoft#28229)
  docs(react-dialog): update DialogTriggerOutsideDialog to include focus behavior (microsoft#28176)
  bugfix: Ensures dialog actions stretches on breakpoints (microsoft#28258)
  applying package updates
  fix: TableHeaderCell should not render button when not sortable (microsoft#28097)
  fix(react-file-type-icons): Map mhtml extension to html icon (microsoft#28112)
  Fix overlapping axis labels on smaller viewports (microsoft#28239)
  useArrowNavigationGroup grid-linear axis (microsoft#28253)
  applying package updates
  fix: Alert example missing aria-label for icon (microsoft#28234)
  Overflow divider fix (microsoft#28011)
  feat(tools): implement `cypress-component-configuration` generator (microsoft#28115)
  chore: migrate to TS 4.7 (microsoft#28067)
  fix(scripts-tasks): make generate-api work in deterministic way (microsoft#28215)
  feat(react-tags): add overflow story (microsoft#28012)
  Structure and slots for SearchBox, using Input as a slot (microsoft#28090)
  feat(tokens): Add/update theme tokens (microsoft#27791)
  feat(react-tags): add a11y role and best practices guide (microsoft#28075)
  fix: Toast intent should always be present in the context (microsoft#28226)
  ...
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.

4 participants