Skip to content

Tag/TagButton init component setup#27102

Merged
YuanboXue-Amber merged 23 commits into
microsoft:masterfrom
micahgodbolt:tag-first-pass
Apr 27, 2023
Merged

Tag/TagButton init component setup#27102
YuanboXue-Amber merged 23 commits into
microsoft:masterfrom
micahgodbolt:tag-first-pass

Conversation

@micahgodbolt

@micahgodbolt micahgodbolt commented Mar 6, 2023

Copy link
Copy Markdown
Member

To break this component down to manageable chunks this PR only focuses on an init setup of types, slots, and basic layout.

Tag

image

TagButton

image

todo:

  1. primary slot
  2. more styles
  3. a11y pass
  4. custom behavior
  5. prop and slot descriptions

@fabricteam

fabricteam commented Mar 6, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against bad96553b2d34ffb3d45c32a44d4883046a15945

@codesandbox-ci

codesandbox-ci Bot commented Mar 6, 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 57a0ed2:

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

@size-auditor

size-auditor Bot commented Mar 6, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: bad96553b2d34ffb3d45c32a44d4883046a15945 (build)

Comment thread packages/react-components/react-tags/src/components/Tag/Tag.types.ts Outdated
Comment thread packages/react-components/react-tags/src/components/Tag/useTag.tsx
Comment thread packages/react-components/react-tags/src/components/Tag/useTagStyles.ts Outdated
Comment thread packages/react-components/react-tags/src/components/TagButton/TagButton.types.ts Outdated
Comment thread packages/react-components/react-tags/src/components/TagButton/renderTagButton.tsx Outdated

@spmonahan spmonahan 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.

Left a few comments, nothing blocking though since this component is early still.

Comment thread packages/react-components/react-tags/src/components/TagButton/useTagButton.tsx Outdated
Comment thread packages/react-components/react-tags/src/components/Tag/useTag.tsx Outdated
Comment thread packages/react-components/react-tags/package.json Outdated
Comment thread packages/react-components/react-tags/etc/react-tags.api.md Outdated
Comment thread packages/react-components/react-tags/etc/react-tags.api.md Outdated
@YuanboXue-Amber YuanboXue-Amber merged commit a3edb5d into microsoft:master Apr 27, 2023
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request May 2, 2023
* master:
  applying package updates
  fix: dont ignore lib, lib-commonjs and lib-amd files (microsoft#27736)
  Add DraggableZone into export map (microsoft#27569)
  Add export for FolderCover.scss (microsoft#27507)
  applying package updates
  Add filetype icon for Microsoft Designer (microsoft#27724)
  ci: add build task back as lint pre-requirement to unblock temporarily import plugin lint rule (microsoft#27729)
  fix(react-charting): add missing dependency to fix lint violations during release (microsoft#27728)
  Added partition for controlled overflow menu (microsoft#27520)
  Azure Theme 8.0: Tag Button variant for DefaultButton and PrimaryButton (microsoft#27186)
  Azure Theme 8.0 dropdown fix for high contrast themes (microsoft#27346)
  chore: enable TS intellisense in cross library projects for better/consistent DX (microsoft#26605)
  chore: enforce files naming to use .styles.ts [cxe-red files] (microsoft#27710)
  chore(v0): enable emit only dts and use new conformance test API to narrow down TS Program (microsoft#27686)
  codeowner update (microsoft#27719)
  Tag/TagButton init component setup (microsoft#27102)
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.

7 participants