Skip to content

fix(react-link): update inverted link styles to use neutral color tokens#34955

Merged
dmytrokirpa merged 2 commits intomicrosoft:masterfrom
dmytrokirpa:fix/inverted-link-colors
Jul 31, 2025
Merged

fix(react-link): update inverted link styles to use neutral color tokens#34955
dmytrokirpa merged 2 commits intomicrosoft:masterfrom
dmytrokirpa:fix/inverted-link-colors

Conversation

@dmytrokirpa
Copy link
Contributor

Previous Behavior

Tokens used for the inverted Link variant did not align with the Figma specification:

  • The code used colorBrandForegroundInverted* tokens
  • Figma used colorNeutralForegroundInvertedLink* tokens

Figma: https://www.figma.com/design/ZhYpaByCXhprTWpF2097yE/Link?node-id=1105-3387&t=98NVbCUII05TFmti-4

New Behavior

The inverted Link variant tokens are now consistent between the component and Figma.

Related Issue(s)

@dmytrokirpa
Copy link
Contributor Author

cc @njsokol

@github-actions
Copy link

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: entire library
1.274 MB
321.014 kB
1.274 MB
321.018 kB
18 B
4 B
react-link
Link
16.728 kB
6.784 kB
16.746 kB
6.785 kB
18 B
1 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-breadcrumb
@fluentui/react-breadcrumb - package
114.212 kB
31.517 kB
react-components
react-components: Button, FluentProvider & webLightTheme
68.79 kB
19.902 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
235.675 kB
68.193 kB
react-components
react-components: FluentProvider & webLightTheme
43.624 kB
14.255 kB
react-message-bar
MessageBar (all components)
24.088 kB
8.96 kB
react-portal-compat
PortalCompatProvider
8.386 kB
2.624 kB
react-timepicker-compat
TimePicker
109.069 kB
36.059 kB
🤖 This report was generated against bbaeb73e492a1b5b61217b516e71731f0f2453fb

@github-actions
Copy link

Pull request demo site: URL

@dmytrokirpa dmytrokirpa merged commit 77dd7de into microsoft:master Jul 31, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Link doesn't work with BackgroundAppearanceProvider

2 participants