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

unexpected scrollable area with right:-Xpx + overflow:hidden #9309

Closed
paulrouget opened this issue Jan 14, 2016 · 0 comments · Fixed by #9405
Closed

unexpected scrollable area with right:-Xpx + overflow:hidden #9309

paulrouget opened this issue Jan 14, 2016 · 0 comments · Fixed by #9405

Comments

@paulrouget
Copy link
Contributor

hzccltqmrh

Even though the container has overflow: hidden, it's still scrollable. Note: overflow content is not visible, only body is visible.

<style>
  body {
    margin: 0;
  }

  .a {
    background: red;
    width: 100%;
    height: 100%;
    position: absolute;
    overflow: hidden;
  }

  .b {
    background: green;
    width: 40px;
    height: 40px;
    position: absolute;
    right: -40px;
  }

</style>

<div class="a">
  <div class="b"></div>
</div>
pcwalton added a commit to pcwalton/servo that referenced this issue Jan 23, 2016
absolutely-positioned flows separately when storing overflow.

This code dates back to the time when absolutely positioned flows were
ignored by all of their ancestors up to the containing block. This
hasn't been true for at least a year.

Closes servo#9306.
Closes servo#9309.
Is a partial fix for servo#9308.
bors-servo pushed a commit that referenced this issue Jan 23, 2016
layout: Remove some bogus code that tried to handle absolutely-positioned flows separately when storing overflow.

This code dates back to the time when absolutely positioned flows were
ignored by all of their ancestors up to the containing block. This
hasn't been true for at least a year.

Closes #9306.
Closes #9309.
Is a partial fix for #9308.

r? @glennw

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9405)
<!-- Reviewable:end -->
pcwalton added a commit to pcwalton/servo that referenced this issue Jan 25, 2016
absolutely-positioned flows separately when storing overflow.

This code dates back to the time when absolutely positioned flows were
ignored by all of their ancestors up to the containing block. This
hasn't been true for at least a year.

Closes servo#9306.
Closes servo#9309.
Is a partial fix for servo#9308.
bors-servo pushed a commit that referenced this issue Jan 25, 2016
layout: Remove some bogus code that tried to handle absolutely-positioned flows separately when storing overflow.

This code dates back to the time when absolutely positioned flows were
ignored by all of their ancestors up to the containing block. This
hasn't been true for at least a year.

Closes #9306.
Closes #9309.
Is a partial fix for #9308.

r? @glennw

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9405)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant