Skip to content

Conversation

@micahgodbolt
Copy link
Member

@micahgodbolt micahgodbolt commented Mar 6, 2023

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
Copy link
Collaborator

fabricteam commented Mar 6, 2023

📊 Bundle size report

🤖 This report was generated against bad96553b2d34ffb3d45c32a44d4883046a15945

@codesandbox-ci
Copy link

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

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

@size-auditor
Copy link

size-auditor bot commented Mar 6, 2023

Asset size changes

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

Baseline commit: bad96553b2d34ffb3d45c32a44d4883046a15945 (build)

Copy link
Contributor

@spmonahan spmonahan left a comment

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.

@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