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

Scrolling improvements #491

Closed
wants to merge 16 commits into from
Closed

Conversation

@pcwalton
Copy link
Contributor

pcwalton commented May 29, 2013

This makes scrolling work a lot better: the entire page is rendered and it's clamped correctly to the boundaries.

r? @metajack

pcwalton added 5 commits May 20, 2013
This is preparation for removing the `servo-` prefixes from the various
Servo components.
…data directly.

Nodes are now parameterized over a "View" type. The particular View type
determines which methods can be called. Layout data accessors and mutators are
only accessible to nodes with a LayoutView. The only way to convert a
`Node<ScriptView>` to a `Node<LayoutView>` is through a transmutation, which is
done at the moment the layout task receives nodes. (This should be factored
better to contain the unsafety.)

We should also lock down DOM node mutation to the ScriptView to forbid data
races, but this patch doesn't do that.

This also reduces coupling between DOM and layout. Soon I would like to move
the DOM into its own crate, and this is a step on the way of doing that.
@pcwalton

This comment has been minimized.

Copy link
Owner Author

pcwalton commented on 83e1bd8 May 28, 2013

r+

This comment has been minimized.

Copy link

metajack replied May 29, 2013

@bors: retry

This comment has been minimized.

Copy link

metajack replied May 30, 2013

@bors: retry

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented on 83e1bd8 May 28, 2013

saw approval from pcwalton
at pcwalton@83e1bd8

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 28, 2013

merging pcwalton/servo/phantom-type = 83e1bd8 into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 28, 2013

pcwalton/servo/phantom-type = 83e1bd8 merged ok, testing candidate = c4fccb4

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 29, 2013

saw approval from pcwalton
at pcwalton@83e1bd8

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

saw approval from pcwalton
at pcwalton@83e1bd8

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

merging pcwalton/servo/phantom-type = 83e1bd8 into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

pcwalton/servo/phantom-type = 83e1bd8 merged ok, testing candidate = 90b01df

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

fast-forwarding master to auto = 90b01df

pcwalton added 4 commits May 28, 2013
…ut interface.

Eventually, the layout interface will be moved along with the DOM into a
separate crate.
Premature abstraction; nothing used the traits polymorphically. Furthermore,
they increased coupling between layout and the DOM.
@pcwalton

This comment has been minimized.

Copy link
Owner Author

pcwalton commented on 0b22336 May 29, 2013

r+

This comment has been minimized.

Copy link

metajack replied May 30, 2013

@bors: retry

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented on 0b22336 May 29, 2013

saw approval from pcwalton
at pcwalton@0b22336

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 29, 2013

merging pcwalton/servo/separation = 0b22336 into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 29, 2013

pcwalton/servo/separation = 0b22336 merged ok, testing candidate = 07461ed

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

saw approval from pcwalton
at pcwalton@0b22336

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

merging pcwalton/servo/separation = 0b22336 into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

pcwalton/servo/separation = 0b22336 merged ok, testing candidate = 473b094

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

fast-forwarding master to auto = 473b094

@pcwalton

This comment has been minimized.

Copy link
Owner Author

pcwalton commented on 6012a3e May 29, 2013

r+

This comment has been minimized.

Copy link

metajack replied May 30, 2013

@bors: retry

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented on 6012a3e May 29, 2013

saw approval from pcwalton
at pcwalton@6012a3e

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 29, 2013

merging pcwalton/servo/display-list-refactor = 6012a3e into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 29, 2013

pcwalton/servo/display-list-refactor = 6012a3e merged ok, testing candidate = 09ce874

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

saw approval from pcwalton
at pcwalton@6012a3e

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

merging pcwalton/servo/display-list-refactor = 6012a3e into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

pcwalton/servo/display-list-refactor = 6012a3e merged ok, testing candidate = 273b6cf

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

fast-forwarding master to auto = 273b6cf

@pcwalton pcwalton closed this May 31, 2013
@pcwalton pcwalton deleted the pcwalton:scrolling-improvements branch May 31, 2013
glennw pushed a commit to glennw/servo that referenced this pull request Jan 16, 2017
Fix regression in the gl -> device API changes.

The viewport was not being set when the render target was None.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/491)
<!-- Reviewable:end -->
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

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