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
style: Make anon boxes account for :visited. #19551
Conversation
Heads up! This PR modifies the following files:
|
r? @heycam |
81effdd
to
07fdc2d
Compare
) -> Arc<ComputedValues> { | ||
// NOTE(emilio): We skip calculating the proper layout parent style |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see the comment in compute_pseudo_element_style_with_inputs
already mentions this, but maybe it's to maintain the concrete example of fieldset?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure thing :)
This should fix one of the test failures of: https://bugzilla.mozilla.org/show_bug.cgi?id=616436
07fdc2d
to
604c520
Compare
@bors-servo r=heycam |
📌 Commit 604c520 has been approved by |
style: Make anon boxes account for :visited. This should fix one of the test failures of: https://bugzilla.mozilla.org/show_bug.cgi?id=616436 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19551) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
This should fix one of the test failures of:
https://bugzilla.mozilla.org/show_bug.cgi?id=616436
This change is