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

Some documents about sticky position are wrong #15230

Merged
merged 1 commit into from
Jun 13, 2022
Merged

Some documents about sticky position are wrong #15230

merged 1 commit into from
Jun 13, 2022

Conversation

MrWcy
Copy link
Contributor

@MrWcy MrWcy commented Apr 22, 2022 •
edited
Loading

"sticky" is not a mixture of static position and fixed position , but relative position and fixed position .
Here's another link to prove it：https://developer.mozilla.org/zh-CN/docs/Web/CSS/position

Summary

Motivation

Supporting details

Related issues

Metadata

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

@MrWcy
Some documents about sticky positioning are wrong
a8dea33 
"sticky" is not a mixture of static position and fixed position , but relative position  and fixed position .
@MrWcy MrWcy requested a review from a team as a code owner April 22, 2022 08:36
@MrWcy MrWcy requested review from rachelandrew and removed request for a team April 22, 2022 08:36
@github-actions github-actions bot added the Content:Learn Learning area docs label Apr 22, 2022
@MrWcy MrWcy changed the title Some documents about sticky positioning are wrong Some documents about sticky position are wrong Apr 22, 2022
@github-actions
Copy link
Contributor

Preview URLs

Flaws

None! 🎉

External URLs

URL: /en-US/docs/Learn/CSS/CSS_layout/Introduction
Title: Introduction to CSS layout
on GitHub

No new external URLs

@sideshowbarker sideshowbarker added the Content:CSS Cascading Style Sheets docs label Apr 24, 2022
estelle
estelle approved these changes Jun 13, 2022
View reviewed changes
Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. Sorry that this PR fell thru the cracks.

@estelle estelle merged commit 908cde1 into mdn:main Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@estelle estelle estelle approved these changes

@rachelandrew rachelandrew Awaiting requested review from rachelandrew rachelandrew was automatically assigned from mdn/yari-content-css

Assignees
No one assigned
Labels
Content:CSS Cascading Style Sheets docs Content:Learn Learning area docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MrWcy @estelle @sideshowbarker