Skip to content

Add Scroll-driven animations (positive)#978

Merged
tantek merged 2 commits into
mainfrom
zcorpan/scroll-driven-animations
Feb 1, 2024
Merged

Add Scroll-driven animations (positive)#978
tantek merged 2 commits into
mainfrom
zcorpan/scroll-driven-animations

Conversation

@zcorpan

@zcorpan zcorpan commented Jan 30, 2024

Copy link
Copy Markdown
Member

Closes #347

@zcorpan

zcorpan commented Jan 30, 2024

Copy link
Copy Markdown
Member Author

Also see Fyrd/caniuse#6956 for caniuse.

@tantek tantek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This position reflects our evaluation per the issue.

One minor nit in the description, it should mention CSS properties first (which are in the draft), and drop "markup". I.e.:

s/Defines an API and markup for/Defines CSS properties and an API for

Comment thread activities.json Outdated
@zcorpan

zcorpan commented Jan 31, 2024

Copy link
Copy Markdown
Member Author

@tantek fixed, and suggested that change to the spec.

@tantek tantek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you and nicely done with the upstream spec fix as well!

@tantek tantek merged commit 275f92c into main Feb 1, 2024
@tantek tantek deleted the zcorpan/scroll-driven-animations branch February 1, 2024 03:17
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.

Scroll-driven Animations

3 participants