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
Bug 1341083: Implement dynamic restyling for display: contents. #15736
Conversation
Heads up! This PR modifies the following files:
|
@bors-servo r=heycam |
📌 Commit 0a2556a has been approved by |
Bug 1341083: Implement dynamic restyling for display: contents. Reviewed upstream by @heycam cc @bholley, didn't end up renaming the `layout_parent` thing, because it made the cascade way more verbose (and difficult to understand IMO) unnecessarily, and you said you were ok-ish with it. <!-- 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/15736) <!-- Reviewable:end -->
💔 Test failed - linux-rel-wpt |
0a2556a
to
39cd718
Compare
@bors-servo r=heycam
|
📌 Commit 39cd718 has been approved by |
Bug 1341083: Implement dynamic restyling for display: contents. Reviewed upstream by @heycam cc @bholley, didn't end up renaming the `layout_parent` thing, because it made the cascade way more verbose (and difficult to understand IMO) unnecessarily, and you said you were ok-ish with it. <!-- 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/15736) <!-- Reviewable:end -->
💔 Test failed - linux-dev |
MozReview-Commit-ID: KimTU2j4V4p Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
39cd718
to
6875c65
Compare
@bors-servo retry
|
@bors-servo r=heycam |
📌 Commit 6875c65 has been approved by |
Bug 1341083: Implement dynamic restyling for display: contents. Reviewed upstream by @heycam cc @bholley, didn't end up renaming the `layout_parent` thing, because it made the cascade way more verbose (and difficult to understand IMO) unnecessarily, and you said you were ok-ish with it. <!-- 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/15736) <!-- Reviewable:end -->
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit 6875c65 has been approved by |
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-gnu-dev, windows-msvc-dev |
Reviewed upstream by @heycam
cc @bholley, didn't end up renaming the
layout_parent
thing, because it made the cascade way more verbose (and difficult to understand IMO) unnecessarily, and you said you were ok-ish with it.This change is