Skip to content

Conversation

@YuanboXue-Amber
Copy link
Contributor

@YuanboXue-Amber YuanboXue-Amber commented Jul 31, 2023

Fix the left over issues on #28368

  1. InteractionTag brand variant had no style change on hover in contrast theme, now it changes text color:
    image

  2. InteractionTag outline variant fills icon on hover and changes icon to brand color
    image

  3. Fix visual issue in Tag overflow story:

    before after
    image image

@YuanboXue-Amber YuanboXue-Amber changed the title fix(react-tags-preview): fix brand InteractionTag hover color fix(react-tags-preview): fix InteractionTag hover styles Jul 31, 2023
@fabricteam
Copy link
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-tags-preview
InteractionTag
34.171 kB
9.199 kB
34.418 kB
9.254 kB
247 B
55 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-tags-preview
Tag
25.962 kB
8.49 kB
react-tags-preview
TagGroup
69.398 kB
20.364 kB
🤖 This report was generated against d4b3feec354590939a340460bea62f4e39d38a74

@codesandbox-ci
Copy link

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

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

@size-auditor
Copy link

size-auditor bot commented Jul 31, 2023

Asset size changes

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

Baseline commit: 9250428d00e7e2909a0e3bae667c6ef0044ea67e (build)

@YuanboXue-Amber YuanboXue-Amber marked this pull request as ready for review July 31, 2023 15:59
@YuanboXue-Amber YuanboXue-Amber requested review from a team as code owners July 31, 2023 15:59
@YuanboXue-Amber YuanboXue-Amber merged commit f06508a into microsoft:master Aug 1, 2023
@YuanboXue-Amber YuanboXue-Amber deleted the tag-color branch August 1, 2023 09:04
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Aug 1, 2023
* master: (37 commits)
  release (microsoft#28696)
  Fixing re-render issue for all charts when empty (microsoft#28321)
  feat(FluentProvider): emit errors on duplicate IDs (microsoft#28670)
  applying package updates
  fix(react-positioning): autoSize causing position update to reach maximum (microsoft#28689)
  fix(react-tags-preview): fix InteractionTag hover styles (microsoft#28686)
  Accordion: export AccordionHeaderProvider (microsoft#28542)
  feat(react-shared-contexts): add AnnounceContext (microsoft#28654)
  Added VR tests for Breadcrumb (microsoft#28653)
  fix(react-menu): use outline for menuItem focus ring (microsoft#28685)
  [Bug]: Tree, vertical spacing of branches and children is inconsistent (microsoft#28681)
  feaTt(react-tree): adds openItems and checkedItems to tree callback data (microsoft#28669)
  applying package updates
  Add mountNode prop to combos (microsoft#28661)
  react-tags-preview: add more vr test (microsoft#28582)
  chore: migrate to nx 16.1.4 (microsoft#28583)
  applying package updates
  chore: improves internal headless signature (microsoft#28651)
  fix: remove margin from icon when ToolbarButton is vertical (microsoft#28658)
  applying package updates
  ...
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.

3 participants