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

Animate contact form on scroll #2511

Merged
merged 4 commits into from
Apr 13, 2023
Merged

Animate contact form on scroll #2511

merged 4 commits into from
Apr 13, 2023

Conversation

kjellr
Copy link
Contributor

@kjellr kjellr commented Apr 11, 2023

PR Summary:

Adds animate-on-scroll classes to the contact form section.

Why are these changes introduced?

Fixes #2509

Visual impact on existing themes

Will animate the contact form section when Reveal sections on scroll is enabled.

11-57-g3ogs-p931i.mp4

Testing steps/scenarios

  1. Visit the demo store
  2. Add a Contact Form section.
  3. Activate Animate on Scroll from within Theme Settings > Motion
  4. Observe that the contact form animates.

Demo links

Checklist

Copy link
Contributor

@ludoboludo ludoboludo left a comment

Choose a reason for hiding this comment

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

Left a comment, otherwise looks good 👍

sections/contact-form.liquid Outdated Show resolved Hide resolved
Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: Ludo <ludo.segura@shopify.com>
@kjellr kjellr merged commit a1b195f into main Apr 13, 2023
@kjellr kjellr deleted the animate-contact-form-on-scroll branch April 13, 2023 12:39
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
* Animate contact form on scroll.

* Update sections/contact-form.liquid

Co-authored-by: Ludo <ludo.segura@shopify.com>

* Update sections/contact-form.liquid

Co-authored-by: Ludo <ludo.segura@shopify.com>

---------

Co-authored-by: Ludo <ludo.segura@shopify.com>
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.

[Contact Form] Add cascading fade in on scroll
3 participants