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

fix: elements with position sticky also form a stacking context #10

Merged
merged 1 commit into from
May 6, 2019

Conversation

roperzh
Copy link
Contributor

@roperzh roperzh commented Jun 22, 2018

According to the MDN web docs, elements with position sticky also
form a stacking context, this adds an additional check to contemplate
this case.

According to the [MDN web docs][1], elements with position `sticky` also
form a stacking context, this adds an additional check to contemplate
this case.

[1]: https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Positioning/Understanding_z_index/The_stacking_context
@roperzh roperzh force-pushed the fix-position-sticky branch from bd9255d to 3611d5f Compare June 22, 2018 11:48
@gwwar
Copy link
Owner

gwwar commented May 6, 2019

Thanks @roperzh! Sorry for the super late response, I need to update my notification settings on GitHub. :D

I'll try and do a latest update pass on rules soon and will include your contribution in the next update!

@gwwar gwwar merged commit 1b2f78e into gwwar:master May 6, 2019
@roperzh
Copy link
Contributor Author

roperzh commented May 6, 2019

haha no worries, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants