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

Website: Added PopupMenu and BottomSheet to mobile docs #10325

Merged
merged 9 commits into from
Aug 30, 2019

Conversation

lynamemi
Copy link
Collaborator

@lynamemi lynamemi commented Aug 30, 2019

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ yarn change

Description of changes

Added two new controls to the Android docs: PopupMenu and BottomSheet.

Focus areas to test

(optional)

Microsoft Reviewers: Open in CodeFlow

@msft-github-bot
Copy link
Contributor

msft-github-bot commented Aug 30, 2019

Component Perf Analysis

No significant results to display.

All results

Scenario Master Ticks PR Ticks Status
BaseButton 795 797
BaseButton (experiments) 1041 1033
DefaultButton 1059 1063
DefaultButton (experiments) 1979 2012
DetailsRow 3310 3382
DetailsRow (fast icons) 3315 3337
DetailsRow without styles 3040 3110
DocumentCardTitle with truncation 32314 32554
MenuButton 1397 1391
MenuButton (experiments) 3632 3621
PrimaryButton 1269 1257
PrimaryButton (experiments) 2124 2047
SplitButton 2974 2840
SplitButton (experiments) 7239 7323
Stack 514 472
Stack with Intrinsic children 1138 1137
Stack with Text children 4370 4395
Text 412 411
Toggle 905 872
Toggle (experiments) 2416 2416
button 67 72

@size-auditor
Copy link

size-auditor bot commented Aug 30, 2019

Asset size changes

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

Baseline commit: 801bf1eaecd186555cf00e38cd294c1bf7914c7d (build)

@kenotron
Copy link
Member

@ecraig12345 - this doc looks okay but probably need your expertise in site.

@lynamemi lynamemi merged commit 03f5cdb into microsoft:master Aug 30, 2019
@lynamemi lynamemi deleted the update-mobile-controls branch August 30, 2019 22:38
@msft-github-bot
Copy link
Contributor

🎉@uifabric/fabric-website@v7.4.3 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.

7 participants