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

style: Remove stale todo about handling scrollbars in viewport units. #18422

Merged
merged 1 commit into from Sep 9, 2017

Conversation

@emilio
Copy link
Member

emilio commented Sep 8, 2017

This is effectively not going to happen, per
w3c/csswg-drafts#1766


This change is Reviewable

This is effectively not going to happen, per
w3c/csswg-drafts#1766
@highfive
Copy link

highfive commented Sep 8, 2017

Heads up! This PR modifies the following files:

  • @bholley: components/style/gecko/media_queries.rs
  • @canaltinova: components/style/gecko/media_queries.rs
@highfive
Copy link

highfive commented Sep 8, 2017

warning Warning warning

  • These commits modify style code, but no tests are modified. Please consider adding a test!
@emilio
Copy link
Member Author

emilio commented Sep 8, 2017

r? @Manishearth (or anyone else)

@highfive highfive assigned Manishearth and unassigned nox Sep 8, 2017
@bholley
Copy link
Contributor

bholley commented Sep 8, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2017

📌 Commit 9a55b1e has been approved by bholley

@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2017

Testing commit 9a55b1e with merge 9c0abfe...

bors-servo added a commit that referenced this pull request Sep 8, 2017
style: Remove stale todo about handling scrollbars in viewport units.

This is effectively not going to happen, per
w3c/csswg-drafts#1766

<!-- 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/18422)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2017

💔 Test failed - linux-rel-wpt

@jdm
Copy link
Member

jdm commented Sep 9, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Sep 9, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Sep 9, 2017

@bors-servo bors-servo merged commit 9a55b1e into servo:master Sep 9, 2017
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@emilio emilio deleted the emilio:stale-todo branch Nov 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants
You can’t perform that action at this time.