Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Update Percy snapshots for the font scaling slider#10858

Closed
luixxiul wants to merge 1 commit into
matrix-org:developfrom
luixxiul:test-appearance-user-settings-tab2
Closed

Update Percy snapshots for the font scaling slider#10858
luixxiul wants to merge 1 commit into
matrix-org:developfrom
luixxiul:test-appearance-user-settings-tab2

Conversation

@luixxiul
Copy link
Copy Markdown
Contributor

@luixxiul luixxiul commented May 11, 2023

Fixes element-hq/element-web#25333

This PR intends to update Percy snapshots for the font scaling slider in order to have Percy take the font scaling panel, instead of the slider since it seems that Percy does not take the align-items and background values into consideration.

type: task

Signed-off-by: Suguru Hirahara luixxiul@users.noreply.github.com

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

This change is marked as an internal change (Task), so will not be included in the changelog.

@github-actions github-actions Bot added Z-Community-PR Issue is solved by a community member's PR T-Task Refactoring, enabling or disabling functionality, other engineering tasks labels May 11, 2023
@luixxiul luixxiul marked this pull request as ready for review May 11, 2023 13:00
@luixxiul luixxiul requested a review from a team as a code owner May 11, 2023 13:00
@kerryarchibald kerryarchibald added the X-Needs-Percy Whether to run Percy screenshot tests in Merge Queue label May 12, 2023
@kerryarchibald kerryarchibald enabled auto-merge May 12, 2023 02:55
@kerryarchibald kerryarchibald added this pull request to the merge queue May 12, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 12, 2023
@luixxiul
Copy link
Copy Markdown
Contributor Author

@luixxiul
Copy link
Copy Markdown
Contributor Author

luixxiul commented May 12, 2023

The PR failed on the merge queue in the same way as #10836 (comment)

@luixxiul
Copy link
Copy Markdown
Contributor Author

@luixxiul luixxiul closed this May 23, 2023
@luixxiul luixxiul deleted the test-appearance-user-settings-tab2 branch May 23, 2023 08:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

T-Task Refactoring, enabling or disabling functionality, other engineering tasks X-Needs-Percy Whether to run Percy screenshot tests in Merge Queue Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Percy does not take snapshots of Font size slider properly

2 participants