Skip to content

Conversation

@sopranopillow
Copy link
Contributor

Previous Behavior

InfoButton's information slot's name is content.

New Behavior

InfoButton's information slot's name is info.

Related Issue(s)

@sopranopillow sopranopillow changed the title chore(react-infobutton): Renaming content to info chore(react-infobutton): Renaming content slot to info Mar 23, 2023
@fabricteam
Copy link
Collaborator

fabricteam commented Mar 23, 2023

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-infobutton
InfoButton
130.601 kB
39.438 kB
130.577 kB
39.439 kB
-24 B
1 B
react-infobutton
InfoLabel
133.901 kB
40.489 kB
133.874 kB
40.488 kB
-27 B
-1 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
67.994 kB
18.395 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
207.4 kB
57.516 kB
react-components
react-components: FluentProvider & webLightTheme
38.819 kB
12.351 kB
react-portal-compat
PortalCompatProvider
6.446 kB
2.185 kB
🤖 This report was generated against 899de68739acdff9e641a1baedec097a6bc07309

@codesandbox-ci
Copy link

codesandbox-ci bot commented Mar 23, 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 c6b2bb0:

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

@size-auditor
Copy link

size-auditor bot commented Mar 23, 2023

Asset size changes

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

Baseline commit: 899de68739acdff9e641a1baedec097a6bc07309 (build)

@sopranopillow sopranopillow requested review from a team as code owners March 24, 2023 17:18
@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 918 906 5000
Button mount 583 599 5000
Field mount 1523 1524 5000
FluentProvider mount 1107 1121 5000
FluentProviderWithTheme mount 292 296 10
FluentProviderWithTheme virtual-rerender 274 275 10
FluentProviderWithTheme virtual-rerender-with-unmount 289 277 10
InfoButton mount 201 199 5000
MakeStyles mount 1369 1368 50000
Persona mount 2087 2090 5000
SpinButton mount 1828 1822 5000

@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@sopranopillow sopranopillow merged commit df9564e into microsoft:master Mar 24, 2023
@sopranopillow sopranopillow deleted the react-infobutton/content-rename branch March 24, 2023 17:47
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Mar 27, 2023
* master: (47 commits)
  fix: Fix vertical alignment of inline Checkbox when checked vs. unchecked (microsoft#27324)
  applying package updates
  Fix: Revert microsoft#26931 change due to new bug it introduced (microsoft#27323)
  chore(react-infobutton): Renaming content slot to info (microsoft#27316)
  applying package updates
  Fix bad release mismatch of react-select (microsoft#27321)
  applying package updates
  Fix react-combobox mismatch after bad release (microsoft#27319)
  docs: add section about media query sorting (microsoft#27307)
  applying package updates
  fix: sync packages and changefiles after bad release (microsoft#27317)
  applying package updates
  chore(react-datepicker-compat): Cleanup types, unused code, and restore utils tests (microsoft#27297)
  [Fluent 2 Theme] Fixing link color in dark theme (microsoft#27203)
  Tree documentation and stories (microsoft#27270)
  feat: Render theme CSS variables in SSR style element (microsoft#27277)
  chore (scripts): update create-package plop templates to use swc/helpers in deps (microsoft#27286)
  fix(DialogContent): adds padding to allow focus visibility (microsoft#27248)
  Adding Card component style hooks to shared context (microsoft#27295)
  Virtualizer: Static scroll hooks and embedded scrollview (microsoft#26985)
  ...
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.

InfoButton: Rename content prop to info and forward children to popover surface

5 participants