Skip to content

Conversation

@kkakroo
Copy link
Contributor

@kkakroo kkakroo commented Jul 17, 2023

Previous Behavior

Incorrect token was assigned to right border primary variant for split button

New Behavior

Correct token assigned to right border primary variant for split button

Related Issue(s)

@kkakroo kkakroo requested review from a team and khmakoto as code owners July 17, 2023 21:02
@fabricteam
Copy link
Collaborator

fabricteam commented Jul 17, 2023

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 601 579 5000
Button mount 294 305 5000
Field mount 1033 1074 5000
FluentProvider mount 639 652 5000
FluentProviderWithTheme mount 66 73 10
FluentProviderWithTheme virtual-rerender 61 61 10
FluentProviderWithTheme virtual-rerender-with-unmount 70 75 10
InfoButton mount 7 15 5000
MakeStyles mount 876 851 50000
Persona mount 1644 1619 5000
SpinButton mount 1327 1316 5000

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 17, 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 8d1ce36:

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

@size-auditor
Copy link

size-auditor bot commented Jul 17, 2023

Asset size changes

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

Baseline commit: ed5b88b0d870a7bdcd890735a9ca2ebf7d7d7265 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Jul 17, 2023

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-button
SplitButton
49.211 kB
12.232 kB
49.183 kB
12.231 kB
-28 B
-1 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-alert
Alert
82.565 kB
21.726 kB
react-button
Button
36.792 kB
9.472 kB
react-button
CompoundButton
43.946 kB
10.952 kB
react-button
MenuButton
40.978 kB
10.669 kB
react-button
ToggleButton
55.074 kB
11.368 kB
react-card
Card - All
89.36 kB
25.284 kB
react-card
Card
83.778 kB
23.685 kB
react-card
CardFooter
9.24 kB
3.907 kB
react-card
CardHeader
11.652 kB
4.741 kB
react-card
CardPreview
10.048 kB
4.254 kB
react-components
react-components: Button, FluentProvider & webLightTheme
65.175 kB
17.915 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
210.68 kB
58.781 kB
react-components
react-components: FluentProvider & webLightTheme
36.409 kB
12.003 kB
react-portal-compat
PortalCompatProvider
6.473 kB
2.196 kB
🤖 This report was generated against ed5b88b0d870a7bdcd890735a9ca2ebf7d7d7265

@fabricteam
Copy link
Collaborator

fabricteam commented Jul 17, 2023

🕵 fluentuiv9 No visual regressions between this PR and main

@khmakoto khmakoto enabled auto-merge (squash) July 19, 2023 23:03
@khmakoto khmakoto merged commit 14804fe into microsoft:master Jul 19, 2023
@kkakroo kkakroo deleted the split-button-border-token branch July 20, 2023 21:52
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jul 25, 2023
* master: (32 commits)
  chore: remove @fluentui/bundle-size (microsoft#28601)
  Breadcrumb UI adjustments (microsoft#28578)
  feat(tools): re-generate react-components.api.md when preparing 1st stable release (microsoft#28561)
  perf(tools): make dependency-mismatch execution 90% faster and ignore */>=9.0.0-alpha versions (microsoft#28597)
  Table/DataGrid: keyboard resizing improvements (microsoft#28493)
  docs(react-tooltip): Add info icon + tooltip story to Tooltip stories (microsoft#28611)
  chore: Updating @fluentui/react-icons to version 2.0.207 (microsoft#28590)
  feat: allSelectedRows and someSelectedRows should be more reliable (microsoft#28577)
  add vr test to react-tags (microsoft#28484)
  applying package updates
  chore: migrate to monosize (microsoft#26826)
  fix(react-conformance): add @swc/helpers to deps instead of tslib as we use swc for transpilation (microsoft#28599)
  fix: MenuItem content should be spaced 12px from the boundary (microsoft#28162)
  feat: implements selection (microsoft#28497)
  bugfix: moves handleBackdropClick from defaultProps to an override (microsoft#28579)
  Fix empty CSS creation (microsoft#28566)
  chore: replace plop with nx within create-* aliases in root package.json (microsoft#28575)
  applying package updates
  fix: High contrast mode hover style icon fixes in react-button components (microsoft#28156)
  SplitButton: updated border right token for primary variant (microsoft#28555)
  ...
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.

[Bug]: Split button primary variant, border right has wrong token color assignment

3 participants