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

css.properties.{scroll-margin,scroll-padding} - Implemented in Safari #8887

Closed
ddbeck opened this issue Jan 28, 2021 · 2 comments
Closed

css.properties.{scroll-margin,scroll-padding} - Implemented in Safari #8887

ddbeck opened this issue Jan 28, 2021 · 2 comments
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS

Comments

@ddbeck
Copy link
Collaborator

ddbeck commented Jan 28, 2021

I'm parking some information from #8103, which can't yet be merged, in this issue.

#8103 was opened on the expectation that the following features are likely to ship in the next release of Safari, since it's in Safari Technology Preview now:

  • css.properties.scroll-margin
  • css.properties.scroll-margin-bottom
  • css.properties.scroll-margin-left
  • css.properties.scroll-margin-right
  • css.properties.scroll-margin-top
  • css.properties.scroll-padding
  • css.properties.scroll-padding-bottom
  • css.properties.scroll-padding-left
  • css.properties.scroll-padding-right
  • css.properties.scroll-padding-top

(see the original PR for more detail, tests, etc.)

But we can't actually add data for these features until either #7396 merges or an actual release of Safari ships with these features.
@ddbeck ddbeck added data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS not ready This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. labels Jan 28, 2021
@ddbeck ddbeck mentioned this issue Jan 28, 2021
Closed
5 tasks
@queengooborg
Copy link
Contributor

Looks like #10129 took care of all the scroll-margin-* properties! I'm surprised that the scroll-padding-* ones weren't included -- looks like a good opportunity for a follow-up.

@queengooborg queengooborg removed the not ready This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. label May 12, 2021
@queengooborg
Copy link
Contributor

The remaining properties were resolved by #12834!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ddbeck @queengooborg