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

Layers overhaul #2003

Closed
pcwalton opened this issue Mar 31, 2014 · 5 comments
Closed

Layers overhaul #2003

pcwalton opened this issue Mar 31, 2014 · 5 comments

Comments

@pcwalton
Copy link
Contributor

@pcwalton pcwalton commented Mar 31, 2014

Layers need to be overhauled to not have a distinction between CompositorLayer and ContainerLayer.

@zwarich zwarich assigned zwarich and unassigned zwarich Jul 10, 2014
@zwarich
Copy link

@zwarich zwarich commented Jul 10, 2014

[Oops, commented in the wrong bug.]

@payload
Copy link
Contributor

@payload payload commented Nov 27, 2014

fixed by 0c2538d?

@frewsxcv
Copy link
Member

@frewsxcv frewsxcv commented Aug 10, 2015

fixed by 0c2538d?

FWIW, at the time of writing, there are still a couple FIXMEs with this issue number in-tree:

> git grep "#2003"
components/layout/block.rs:            // FIXME(#2003, pcwalton): The max is taken here so that you can scroll the page, but
components/script/script_task.rs:        // FIXME(#2003, pcwalton): This is pretty bogus when multiple layers are involved.
@mrobinson
Copy link
Member

@mrobinson mrobinson commented Aug 26, 2015

I think the block.rs comment might be invalid now, because we have a different scrolling model in the compositor now. The comment in script_task.rs looks valid, but I don't think it's necessarily related to this bug any longer.

@glennw
Copy link
Member

@glennw glennw commented Feb 11, 2017

No longer relevant.

@glennw glennw closed this Feb 11, 2017
@pyfisch pyfisch mentioned this issue Jan 4, 2018
4 of 13 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.