Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add few small toolbar improvements #25468

Merged
merged 2 commits into from
Nov 1, 2022

Conversation

chpalac
Copy link
Contributor

@chpalac chpalac commented Nov 1, 2022

Overview

Adding few small Toolbar improvements:

  • Fix onCheckedValuesChange not being called
  • Add aria-label for overflow button items.
  • Add more meaningful value for the controlled toggle button example.

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 1, 2022

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
62.94 kB
17.663 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
189.603 kB
52.909 kB
react-components
react-components: FluentProvider & webLightTheme
33.446 kB
11.033 kB
react-portal-compat
PortalCompatProvider
5.857 kB
1.978 kB
🤖 This report was generated against 51eaa880e902bb7b24739eff1d3c447e6edc6766

@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 1, 2022

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 8a7a53a:

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

@size-auditor
Copy link

size-auditor bot commented Nov 1, 2022

Asset size changes

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

Baseline commit: 51eaa880e902bb7b24739eff1d3c447e6edc6766 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 1, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1525 1556 5000
Button mount 1165 1185 5000
FluentProvider mount 1898 1873 5000
FluentProviderWithTheme mount 780 785 10
FluentProviderWithTheme virtual-rerender 722 689 10
FluentProviderWithTheme virtual-rerender-with-unmount 739 730 10
MakeStyles mount 2232 2294 50000
SpinButton mount 3000 3095 5000

@chpalac chpalac merged commit abfb64d into microsoft:master Nov 1, 2022
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Nov 2, 2022
* master: (23 commits)
  fix(docsite-v9): move theme picker under component title so it can be visible on embedded pages (microsoft#25385)
  applying package updates
  feat: Implement child render function for DataGrid rows (microsoft#25476)
  fix(useTable): sort should adapt to enhanced row types (microsoft#25487)
  applying package updates
  feat: positioning should happen out of React lifecycle (microsoft#25456)
  applying package updates
  chore(react-link): migrate to new package structure (microsoft#25471)
  chore(react-input): migrate to new package structure (microsoft#25469)
  fix glob pattern for syncpack (microsoft#25465)
  chore(react-label): migrate to new package structure (microsoft#25470)
  chore: bump Griffel to latest (microsoft#25412)
  chore: add few small toolbar improvements (microsoft#25468)
  docs: fix small typos (microsoft#25464)
  chore: fix dependencies in @fluentui/react-toolbar (microsoft#25466)
  feat: v0 menu style migration from v9 (microsoft#25012)
  applying package updates
  Update List to render children on first render() call (microsoft#25331)
  Scaffold react-skeleton package (microsoft#25435)
  fix(public-docsite): Changing crossplatform urls to use 'cross' instead of 'crossplatform' (microsoft#25437)
  ...
NotWoods pushed a commit to NotWoods/fluentui that referenced this pull request Nov 18, 2022
* chore: add few small toolbar improvements

* chore: add changes
Hotell pushed a commit to Hotell/fluentui that referenced this pull request Feb 9, 2023
* chore: add few small toolbar improvements

* chore: add changes
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.

4 participants