Skip to content

Conversation

@sopranopillow
Copy link
Contributor

After some offline discussion, we've decided to rename InfoTip to InfoIcon to better align with InfoButton's name.

@sopranopillow sopranopillow self-assigned this Jun 8, 2023
@sopranopillow sopranopillow requested a review from a team as a code owner June 8, 2023 18:58
@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 578 580 5000
Button mount 297 300 5000
Field mount 1006 1025 5000
FluentProvider mount 655 654 5000
FluentProviderWithTheme mount 81 76 10
FluentProviderWithTheme virtual-rerender 63 65 10
FluentProviderWithTheme virtual-rerender-with-unmount 65 75 10
InfoButton mount 9 16 5000
MakeStyles mount 846 868 50000
Persona mount 1567 1567 5000
SpinButton mount 1278 1257 5000

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jun 8, 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 2f8cd58:

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

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
64.935 kB
17.918 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
211.151 kB
58.715 kB
react-components
react-components: FluentProvider & webLightTheme
36.168 kB
11.964 kB
react-infobutton
InfoButton
132.502 kB
40.405 kB
react-infobutton
InfoLabel
135.967 kB
41.466 kB
react-portal-compat
PortalCompatProvider
6.473 kB
2.196 kB
🤖 This report was generated against 43e7e505483ad342e727a354f9b4ffb36ae9682a

@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@sopranopillow sopranopillow changed the title chore(react-infobutton): Rename InfoTip to InfoButton chore(react-infobutton): Rename InfoTip to InfoIcon Jun 8, 2023
@size-auditor
Copy link

size-auditor bot commented Jun 8, 2023

Asset size changes

⚠️ Insufficient baseline data to detect size changes

Unable to find bundle size details for Baseline commit: 43e7e50

Possible causes

  • The baseline build 43e7e50 is broken
  • The Size Auditor run for the baseline build 43e7e50 was not triggered

Recommendations

  • Please merge your branch for this Pull request with the latest master build and commit your changes once again

@sopranopillow sopranopillow merged commit 91b9ce0 into microsoft:master Jun 8, 2023
@sopranopillow sopranopillow deleted the rename-infotip branch June 8, 2023 22:11
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 13, 2023
* master:
  applying package updates
  doc: Remove wrong recommendation from input (microsoft#28168)
  fix: Outline color now respects OS force colors settings (microsoft#28182)
  Datepicker onCalendarOpenStateChange prop (microsoft#28136)
  feat(react-file-type-icons): Add getFileTypeIconAsUrl utility (microsoft#27336)
  applying package updates
  [FIX] Updates selection dot and label positioning for ChoiceGroupOption when an Image or Icon is present. (microsoft#28173)
  chore(react-infobutton): Rename InfoTip to InfoIcon (microsoft#28177)
  fix(react-datepicker-compat): Make DatePicker Compat stories SSR safe (microsoft#28017)
  Added `tab` focus mode to the Breadcrumb (microsoft#28046)
  docs: Add usable documentation (microsoft#28155)
  applying package updates
  Fix up rebase code (microsoft#27864)
  fix: ContextualMenu indexing in sections (microsoft#28127)
  applying package updates
  fix: Layer sets focus visible classname for its FocusRectsProvider (microsoft#28157)
  docs(react-infobutton): Update stories to show better guidance (microsoft#28144)
  fix(react-avatar): Do not render the image when src prop is undefined (microsoft#28146)
  feat(react-datepicker-compat): Add null to value prop and handle controlled cases correctly (microsoft#28056)
@msft-fluent-ui-bot
Copy link
Collaborator

🎉@fluentui/[email protected] has been released which incorporates this pull request.:tada:

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

🎉@fluentui/[email protected] has been released which incorporates this pull request.:tada:

Handy links:

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.

4 participants