Skip to content

Use dpr and window size from binding for accessibility guide#39577

Merged
jonahwilliams merged 1 commit intoflutter:masterfrom
jonahwilliams:fix_dpr_value
Sep 2, 2019
Merged

Use dpr and window size from binding for accessibility guide#39577
jonahwilliams merged 1 commit intoflutter:masterfrom
jonahwilliams:fix_dpr_value

Conversation

@jonahwilliams
Copy link
Copy Markdown
Contributor

Description

Previously the contrast test logic assumed 800x600 and a dpr of 3.0. Remove the hardcoded values and replace with a lookup on the binding's window to support different screen sizes and resolutions.

@fluttergithubbot fluttergithubbot added a: tests "flutter test", flutter_test, or one of our tests framework flutter/packages/flutter repository. See also f: labels. labels Aug 30, 2019
@jonahwilliams
Copy link
Copy Markdown
Contributor Author

cc @johnsonmh, can you double check that this works for you?

@jonahwilliams jonahwilliams changed the title Use dpr and window size from binding Use dpr and window size from binding for accessibility guide Aug 30, 2019
Copy link
Copy Markdown
Contributor

@johnsonmh johnsonmh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, this solution works for me, thanks for the quick PR Jonah!

@jonahwilliams jonahwilliams merged commit 7bc4074 into flutter:master Sep 2, 2019
@jonahwilliams jonahwilliams deleted the fix_dpr_value branch September 2, 2019 06:57
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

a: tests "flutter test", flutter_test, or one of our tests framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants