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

The outline and the filled part of MicPulseOff were both filled. Fixing. #23021

Merged
merged 9 commits into from
May 18, 2022

Conversation

sfilipi
Copy link
Member

@sfilipi sfilipi commented May 16, 2022

Current Behavior

MicPulseOff path for outline and filled is the same

Fixes #
Fixes the MicPulseOff outline path per what is in the Figma file.

image

@sfilipi sfilipi requested a review from a team as a code owner May 16, 2022 21:20
@msft-fluent-ui-bot msft-fluent-ui-bot added the Fluent UI react-northstar (v0) Work related to Fluent UI V0 label May 16, 2022
@codesandbox-ci
Copy link

codesandbox-ci bot commented May 16, 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 fbf8388:

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

@size-auditor
Copy link

size-auditor bot commented May 16, 2022

Asset size changes

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

Baseline commit: 9c431fb57b2c1ea13558e81e24ec7f3498359f2f (build)

@fabricteam
Copy link
Collaborator

fabricteam commented May 16, 2022

📊 Bundle size report

🤖 This report was generated against 9c431fb57b2c1ea13558e81e24ec7f3498359f2f

@fabricteam
Copy link
Collaborator

fabricteam commented May 16, 2022

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
ListMinimalPerf.default 536 485 1.11:1
AvatarMinimalPerf.default 203 186 1.09:1
GridMinimalPerf.default 337 314 1.07:1
ListNestedPerf.default 588 551 1.07:1
AlertMinimalPerf.default 273 258 1.06:1
ChatDuplicateMessagesPerf.default 300 283 1.06:1
PortalMinimalPerf.default 180 170 1.06:1
VideoMinimalPerf.default 669 633 1.06:1
LoaderMinimalPerf.default 717 681 1.05:1
RadioGroupMinimalPerf.default 463 439 1.05:1
TextAreaMinimalPerf.default 509 485 1.05:1
ChatMinimalPerf.default 756 729 1.04:1
ReactionMinimalPerf.default 379 364 1.04:1
SegmentMinimalPerf.default 370 356 1.04:1
IconMinimalPerf.default 614 590 1.04:1
AttachmentMinimalPerf.default 161 156 1.03:1
ButtonSlotsPerf.default 565 549 1.03:1
EmbedMinimalPerf.default 4154 4040 1.03:1
LayoutMinimalPerf.default 360 351 1.03:1
ProviderMinimalPerf.default 418 406 1.03:1
TreeMinimalPerf.default 834 811 1.03:1
TreeWith60ListItems.default 175 170 1.03:1
AnimationMinimalPerf.default 557 544 1.02:1
BoxMinimalPerf.default 347 340 1.02:1
DialogMinimalPerf.default 796 781 1.02:1
LabelMinimalPerf.default 387 378 1.02:1
PopupMinimalPerf.default 645 630 1.02:1
TableManyItemsPerf.default 1995 1963 1.02:1
ButtonMinimalPerf.default 163 162 1.01:1
CardMinimalPerf.default 575 572 1.01:1
CheckboxMinimalPerf.default 2792 2771 1.01:1
DropdownManyItemsPerf.default 694 684 1.01:1
DropdownMinimalPerf.default 3087 3054 1.01:1
InputMinimalPerf.default 1285 1271 1.01:1
ItemLayoutMinimalPerf.default 1169 1158 1.01:1
SkeletonMinimalPerf.default 350 346 1.01:1
StatusMinimalPerf.default 684 676 1.01:1
TextMinimalPerf.default 345 343 1.01:1
ChatWithPopoverPerf.default 387 388 1:1
DatepickerMinimalPerf.default 6343 6316 1:1
DividerMinimalPerf.default 349 349 1:1
ImageMinimalPerf.default 362 363 1:1
MenuMinimalPerf.default 868 870 1:1
SplitButtonMinimalPerf.default 4448 4443 1:1
TableMinimalPerf.default 404 403 1:1
ButtonOverridesMissPerf.default 1530 1539 0.99:1
FlexMinimalPerf.default 272 274 0.99:1
FormMinimalPerf.default 382 384 0.99:1
HeaderSlotsPerf.default 730 736 0.99:1
MenuButtonMinimalPerf.default 1753 1773 0.99:1
ProviderMergeThemesPerf.default 1305 1323 0.99:1
RefMinimalPerf.default 244 246 0.99:1
SliderMinimalPerf.default 1712 1726 0.99:1
CustomToolbarPrototype.default 2801 2818 0.99:1
ToolbarMinimalPerf.default 937 944 0.99:1
HeaderMinimalPerf.default 333 341 0.98:1
ListCommonPerf.default 622 633 0.98:1
AttachmentSlotsPerf.default 1110 1139 0.97:1
CarouselMinimalPerf.default 475 490 0.97:1
TooltipMinimalPerf.default 1128 1166 0.97:1
RosterPerf.default 1087 1131 0.96:1
AccordionMinimalPerf.default 139 149 0.93:1
ListWith60ListItems.default 578 621 0.93:1

Co-authored-by: ling1726 <lingfangao@hotmail.com>
@jurokapsiar jurokapsiar reopened this May 18, 2022
@microsoft microsoft deleted a comment from DustyTheBot May 18, 2022
@jurokapsiar jurokapsiar reopened this May 18, 2022
@jurokapsiar
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@microsoft microsoft deleted a comment from DustyTheBot May 18, 2022
@jurokapsiar jurokapsiar merged commit 8691a36 into microsoft:master May 18, 2022
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
…ng. (microsoft#23021)

* the outline and the filled part were both filled. Fixing.

* editing the changelog

* should not set the fill

* the format looks right to me, but maybe the '.' is throwing checking off

* Update packages/fluentui/CHANGELOG.md

Co-authored-by: ling1726 <lingfangao@hotmail.com>

* Update CHANGELOG.md

* Update packages/fluentui/CHANGELOG.md

* Update CHANGELOG.md

* fix changelog

Co-authored-by: Juraj Kapsiar <jurokapsiar@gmail.com>
Co-authored-by: ling1726 <lingfangao@hotmail.com>
Co-authored-by: Miroslav Stastny <miroslav.stastny@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fluent UI react-northstar (v0) Work related to Fluent UI V0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants