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.

Sign up for GitHub

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

Jump to bottom

LibWeb: Abspos element is painted above element with higher z-index #21137

Closed
awesomekling opened this issue Sep 18, 2023 · 0 comments · Fixed by #22543
Closed

LibWeb: Abspos element is painted above element with higher z-index #21137

awesomekling opened this issue Sep 18, 2023 · 0 comments · Fixed by #22543
Labels
bug Something isn't working has-repro We have a way to reproduce this bug.

Comments

@awesomekling
Copy link
Contributor

Repro:

<!doctype html><style>
* {
    outline: 1px solid black;
    font-size: 60px;
}
body {
    /* NOTE: This is flex in order to force div.text to create a stacking context. */
    display: flex;
}
.abspos {
    position: absolute;
    width: 100px;
    height: 100px;
    background: gray;
	z-index: 0;
}
.text {
    z-index: 1;
}
</style><body><div class="abspos"></div><div class="text">hello friends
@awesomekling awesomekling added bug Something isn't working has-repro We have a way to reproduce this bug. labels Sep 18, 2023
@ADKaster ADKaster mentioned this issue Sep 23, 2023
Open
@Lubrsi Lubrsi mentioned this issue Oct 5, 2023
Closed
kalenikaliaksandr added a commit to kalenikaliaksandr/serenity that referenced this issue Jan 2, 2024
@kalenikaliaksandr
LibWeb: Do not require box to be positioned to create stacking context
de79408 
Instead of implementing stacking context painting exactly as it is
defined in CSS2.2 "Appendix E. Elaborate description of Stacking
Contexts" we need to account for changes in the latest standards where
a box can establish a stacking context without being positioned, for
example, by having an opacity different from 1.

Fixes SerenityOS#21137
kalenikaliaksandr added a commit to kalenikaliaksandr/serenity that referenced this issue Jan 2, 2024
@kalenikaliaksandr
LibWeb: Do not require box to be positioned to create stacking context
159a7db 
Instead of implementing stacking context painting order exactly as it
is defined in CSS2.2 "Appendix E. Elaborate description of Stacking
Contexts" we need to account for changes in the latest standards where
a box can establish a stacking context without being positioned, for
example, by having an opacity different from 1.

Fixes SerenityOS#21137
@kalenikaliaksandr kalenikaliaksandr mentioned this issue Jan 2, 2024
Merged
kalenikaliaksandr added a commit to kalenikaliaksandr/serenity that referenced this issue Jan 2, 2024
@kalenikaliaksandr
LibWeb: Do not require box to be positioned to create stacking context
636100a 
Instead of implementing stacking context painting order exactly as it
is defined in CSS2.2 "Appendix E. Elaborate description of Stacking
Contexts" we need to account for changes in the latest standards where
a box can establish a stacking context without being positioned, for
example, by having an opacity different from 1.

Fixes SerenityOS#21137
kalenikaliaksandr added a commit to kalenikaliaksandr/serenity that referenced this issue Jan 2, 2024
@kalenikaliaksandr
LibWeb: Do not require box to be positioned to create stacking context
c614ee1 
Instead of implementing stacking context painting order exactly as it
is defined in CSS2.2 "Appendix E. Elaborate description of Stacking
Contexts" we need to account for changes in the latest standards where
a box can establish a stacking context without being positioned, for
example, by having an opacity different from 1.

Fixes SerenityOS#21137
kalenikaliaksandr added a commit that referenced this issue Jan 2, 2024
@kalenikaliaksandr
LibWeb: Do not require box to be positioned to create stacking context
49fcc5d 
Instead of implementing stacking context painting order exactly as it
is defined in CSS2.2 "Appendix E. Elaborate description of Stacking
Contexts" we need to account for changes in the latest standards where
a box can establish a stacking context without being positioned, for
example, by having an opacity different from 1.

Fixes #21137
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working has-repro We have a way to reproduce this bug.
Projects
No open projects
Ladybird
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

LibWeb: Do not require box to be positioned to create stacking context kalenikaliaksandr/serenity
1 participant
@awesomekling