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

Added 4 active-selection-* tests #24710

Closed

Conversation

TalbotG
Copy link
Contributor

@TalbotG TalbotG commented Jul 23, 2020

active-selection-021.html
reference/active-selection-021-ref.html

active-selection-024.html
reference/active-selection-024-ref.html

active-selection-031.html
reference/active-selection-031-ref.html

active-selection-045.html

support/100x100-red.png

This is a followup to
#20931

@TalbotG TalbotG assigned frivoal and fantasai and unassigned plinss Jul 23, 2020
@TalbotG
Copy link
Contributor Author

TalbotG commented Jul 23, 2020

:( I am going to try again with 2 smaller PRs.

@TalbotG
Copy link
Contributor Author

TalbotG commented Jul 23, 2020

active-selection-045.html is most likely the test
that makes the check wpt-chrome-dev-stability fails.
Now, the only declaration that could affect Chrome is
font-size: 300px .
I am carefully checking again this in
#24723
and if the check wpt-chrome-dev-stability fails
again over there, then I will try again with
font-size: 150px .

When I try active-selection-045.html with
Chromium 86.0.4211.0 (dev build)
with a font size of 300px,
I see/get no problem at all.

@TalbotG
Copy link
Contributor Author

TalbotG commented Jul 24, 2020

I must close this PR and delete this branch now that
#24713
#24714
have merged.

@TalbotG TalbotG closed this Jul 24, 2020
@TalbotG TalbotG deleted the CSS4Pseudo-GT-PR27 branch July 24, 2020 17:39
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.

5 participants