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

Button: Adding visual regression tests #9159

Merged
merged 6 commits into from
May 29, 2019

Conversation

khmakoto
Copy link
Member

@khmakoto khmakoto commented May 20, 2019

Pull request checklist

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

Description of changes

This PR adds VR tests for the new Button component and all of its variants.

Focus areas to test

(optional)

Microsoft Reviewers: Open in CodeFlow

@micahgodbolt micahgodbolt mentioned this pull request May 21, 2019
37 tasks
@msft-github-bot
Copy link
Contributor

msft-github-bot commented May 21, 2019

Component perf results:

Scenario Target branch avg total (ms) PR avg total (ms) Target branch avg per item (ms) PR avg per item (ms) Is significant change Is regression
PrimaryButton 96.886 94.502 0.969 0.945 false false
BaseButton 41.504 43.327 0.415 0.433 false false
NewButton 130.014 ... 1.300 ... ... ...
button 7.448 8.049 0.074 0.080 false false
DetailsRows without styles 227.074 220.330 2.271 2.203 false false
DetailsRows 252.399 222.989 2.524 2.230 true false
Toggles 63.108 50.515 0.631 0.505 true false
NewToggle 83.674 79.289 0.837 0.793 true false
DocumentCardTitle with truncation 33.929 30.758 0.339 0.308 true false
DefaultButton ... 88.235 ... 0.882 ... ...
MenuButton ... 115.008 ... 1.150 ... ...
SplitButton ... 213.701 ... 2.137 ... ...
NewDefaultButton ... 102.973 ... 1.030 ... ...
NewPrimaryButton ... 102.821 ... 1.028 ... ...
NewMenuButton ... 182.555 ... 1.826 ... ...
NewSplitButton ... 354.059 ... 3.541 ... ...

@msft-github-bot
Copy link
Contributor

Hello @khmakoto!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msft-github-bot) and give me an instruction to get started! Learn more here.

@msft-github-bot msft-github-bot merged commit bb3eb04 into microsoft:fabric-7 May 29, 2019
@khmakoto khmakoto deleted the buttonVRTests branch May 29, 2019 23:56
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants