Skip to content

Add visual regression test for TilesList#26969

Merged
spmonahan merged 2 commits into
microsoft:masterfrom
ThomasMichon:tiles-list-stories
Mar 6, 2023
Merged

Add visual regression test for TilesList#26969
spmonahan merged 2 commits into
microsoft:masterfrom
ThomasMichon:tiles-list-stories

Conversation

@ThomasMichon

Copy link
Copy Markdown
Member

Added a new visual regression test for TilesList to ensure that changes to List don't break its render behavior, which requires an upfront measurement of the content width before rendering can occur.

@ThomasMichon
ThomasMichon requested a review from a team as a code owner February 25, 2023 23:22
@fabricteam

fabricteam commented Feb 25, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against efb2274e37c0237f3a10a4630b2696a9880f6398

@codesandbox-ci

codesandbox-ci Bot commented Feb 25, 2023

Copy link
Copy Markdown

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 3f2b4db:

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

@size-auditor

size-auditor Bot commented Feb 25, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: efb2274e37c0237f3a10a4630b2696a9880f6398 (build)

Comment thread apps/vr-tests/src/stories/TilesList.stories.tsx
Comment thread apps/vr-tests/src/stories/TilesList.stories.tsx
Comment thread apps/vr-tests/src/stories/TilesList.stories.tsx
@spmonahan
spmonahan merged commit 7883c8f into microsoft:master Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants