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
Move some font code around #20047
Move some font code around #20047
Conversation
Heads up! This PR modifies the following files:
|
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.
rs=me
Just for the record, the KeywordInfo change is because of:
15:20 <emilio> nox: why did keywordinfo become generic?
15:21 <nox> emilio: Because I plan to make NonNegativeLength animate as Length.
15:22 <nox> emilio: TL;DR if I want to continue #20040 I need to clean a dozen rabbit holes.
TravisCI shows several build errors. |
Oops, I only ran |
04155bc
to
ee8690a
Compare
@bors-servo r=emilio |
📌 Commit ee8690a has been approved by |
Move some font code around The only notable change is that KeywordInfo is now generic. <!-- 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/20047) <!-- Reviewable:end -->
💔 Test failed - linux-dev |
ee8690a
to
d8c43ac
Compare
@bors-servo r=emilio |
📌 Commit d8c43ac has been approved by |
Move some font code around The only notable change is that KeywordInfo is now generic. <!-- 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/20047) <!-- 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 |
The only notable change is that KeywordInfo is now generic.
This change is