Skip to content

feat(react-tags): Add TagGroup with context#27886

Merged
YuanboXue-Amber merged 13 commits into
microsoft:masterfrom
YuanboXue-Amber:tag-temp
May 18, 2023
Merged

feat(react-tags): Add TagGroup with context#27886
YuanboXue-Amber merged 13 commits into
microsoft:masterfrom
YuanboXue-Amber:tag-temp

Conversation

@YuanboXue-Amber

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

Copy link
Copy Markdown
Contributor

Add TagGroup component that wraps around Tag/TagButton, and providing context.
Currently there's only size in TagGroupContext

usage:

<TagGroup size="small">
  <Tag>Tag 1</Tag>
  <TagButton>Tag 2</TagButton>
</TagGroup>

@YuanboXue-Amber
YuanboXue-Amber marked this pull request as ready for review May 17, 2023 14:04
@YuanboXue-Amber
YuanboXue-Amber requested review from a team as code owners May 17, 2023 14:04
@fabricteam

fabricteam commented May 17, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-tags
Tag
20.637 kB
7.332 kB
20.916 kB
7.449 kB
279 B
117 B

🤖 This report was generated against dc86a4ef3e06e3828ccf86961f23289a50718faf

@codesandbox-ci

codesandbox-ci Bot commented May 17, 2023

Copy link
Copy Markdown

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 a9d53fd:

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

@size-auditor

size-auditor Bot commented May 17, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: dc86a4ef3e06e3828ccf86961f23289a50718faf (build)

@bsunderhus bsunderhus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the proper jsx comments on top of the render file otherwise getSlotsNext won't work properly.

Besides this LGTM.

Comment thread packages/react-components/react-tags/src/components/TagGroup/renderTagGroup.tsx Outdated
YuanboXue-Amber and others added 3 commits May 18, 2023 10:19

@bsunderhus bsunderhus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💪🏻✅

YuanboXue-Amber and others added 3 commits May 18, 2023 10:52
@YuanboXue-Amber
YuanboXue-Amber merged commit 31930f6 into microsoft:master May 18, 2023
@YuanboXue-Amber
YuanboXue-Amber deleted the tag-temp branch May 18, 2023 09:19
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request May 18, 2023
* master: (24 commits)
  chore(react-tabster): upgrade tabster to v4.4.2 (microsoft#27540)
  feat(react-tags): Add TagGroup with context (microsoft#27886)
  applying package updates
  fix(react-infobutton): Add aria-owns to InfoLabel (microsoft#27834)
  fix(recipes-react-components): Add a FluentProvider to the local storybook (microsoft#27746)
  chore: update RFC template (microsoft#27880)
  applying package updates
  feat: implement Toaster offset (microsoft#27854)
  feat(react-drawer): create DrawerFooter component (microsoft#27583)
  Make getKey and selection props mutually exclusive (microsoft#24048)
  Added MIGRATION.md to the Breadcrumb (microsoft#27846)
  update Github CODEOWNERS file (microsoft#27849)
  feat(react-tags): make basic Tag a button instead of div (microsoft#27858)
  chore: add test-ssr script to v9 packages (microsoft#27690)
  chore(react-tree): exports TreeItemAside unstable (microsoft#27856)
  bugfix(react-dialog): removes unnecessary grid gaps (microsoft#27845)
  applying package updates
  fix(react-textarea): Don't remove outline when filled and disabled and apply correct disabled color to text (microsoft#27837)
  feat: Implement limit for toast stacking (microsoft#27848)
  Update README.md for fluent 2 theme to include import instructions (microsoft#27847)
  ...
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request May 18, 2023
* feat/drawer-header: (24 commits)
  chore(react-tabster): upgrade tabster to v4.4.2 (microsoft#27540)
  feat(react-tags): Add TagGroup with context (microsoft#27886)
  applying package updates
  fix(react-infobutton): Add aria-owns to InfoLabel (microsoft#27834)
  fix(recipes-react-components): Add a FluentProvider to the local storybook (microsoft#27746)
  chore: update RFC template (microsoft#27880)
  applying package updates
  feat: implement Toaster offset (microsoft#27854)
  feat(react-drawer): create DrawerFooter component (microsoft#27583)
  Make getKey and selection props mutually exclusive (microsoft#24048)
  fix: move style override to outside the component
  Added MIGRATION.md to the Breadcrumb (microsoft#27846)
  update Github CODEOWNERS file (microsoft#27849)
  feat(react-tags): make basic Tag a button instead of div (microsoft#27858)
  chore: add test-ssr script to v9 packages (microsoft#27690)
  chore(react-tree): exports TreeItemAside unstable (microsoft#27856)
  bugfix(react-dialog): removes unnecessary grid gaps (microsoft#27845)
  applying package updates
  fix(react-textarea): Don't remove outline when filled and disabled and apply correct disabled color to text (microsoft#27837)
  feat: Implement limit for toast stacking (microsoft#27848)
  ...
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.

5 participants