Skip to content

fix(react-tags-preview): remove disabled InteractionTag hover color under WHCM#29275

Merged
YuanboXue-Amber merged 2 commits into
microsoft:masterfrom
YuanboXue-Amber:tag-hc
Sep 25, 2023
Merged

fix(react-tags-preview): remove disabled InteractionTag hover color under WHCM#29275
YuanboXue-Amber merged 2 commits into
microsoft:masterfrom
YuanboXue-Amber:tag-hc

Conversation

@YuanboXue-Amber

@YuanboXue-Amber YuanboXue-Amber commented Sep 25, 2023

Copy link
Copy Markdown
Contributor

fix #29224.

PR #29035 added WHCM background color on hover to InteractionTag, increasing its contrast. However it changed disabled InteractionTag as well.
This PR removes hover background for disabled InteractionTag, and keep the hover background only for normal InteractionTag.

Before:
imageimage

After (no hover style):
Screenshot 2023-09-25 at 13 44 15

Normal InteractionTag stays the same:
imageimage

@YuanboXue-Amber
YuanboXue-Amber marked this pull request as ready for review September 25, 2023 11:45
@YuanboXue-Amber
YuanboXue-Amber requested review from a team as code owners September 25, 2023 11:45
@codesandbox-ci

codesandbox-ci Bot commented Sep 25, 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 7b5e285:

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

@fabricteam

fabricteam commented Sep 25, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-tags-preview
TagGroup
72.851 kB
21.693 kB
72.793 kB
21.685 kB
-58 B
-8 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-tags-preview
InteractionTag
14.356 kB
5.746 kB
react-tags-preview
Tag
29.146 kB
9.108 kB
🤖 This report was generated against cb6dfbf4f9d0f7839fbecd5450221477aadb5987

@size-auditor

size-auditor Bot commented Sep 25, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: a56c408d4ed860a48e22780289e5c4a00ebec6c4 (build)

@fabricteam

fabricteam commented Sep 25, 2023

Copy link
Copy Markdown
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@YuanboXue-Amber
YuanboXue-Amber enabled auto-merge (squash) September 25, 2023 13:14
@YuanboXue-Amber
YuanboXue-Amber merged commit e90de0d into microsoft:master Sep 25, 2023
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Sep 25, 2023
…l-to-composition

* master:
  feat: FlatTree supports navigation without useHeadlessFlatTree (microsoft#29091)
  fix(react-card): sync controllable state for selectable cards (microsoft#29276)
  fix (microsoft#29275)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Tag v9 Tag component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disabled tag has hover state in windows high contrast mode

3 participants