-
Notifications
You must be signed in to change notification settings - Fork 24
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
Coming soon section flow revamp #706
Conversation
- In the `SectionContent` component, we are adding a new optional `id` prop. - In the `SectionContent` component, we are tracking url params and looking for `target`. - If the value of the target param matches the passed value in the `id` prop, we'll temporarily add a blink animation. - We are also updating the coming soon section title as per this ticket: https://jira.newfold.com/projects/PRESS1/issues/PRESS1-161
Accidentally pushed my local's cypress configs
1 replays were recorded for 58d39b8. 1 Failed
|
This is part of this PR: bluehost/bluehost-wordpress-plugin#706 We are changing the coming soon section title from 'Maintenance Mode' to 'Site Status: ${status}'.
Coming Soon Module PR: newfold-labs/wp-module-coming-soon#13 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the blink is a bit much/long. I counted 6 flashes when the page loaded. Each blink animation transitions the color twice and then we repeat 3 times. I think we could just keyframe one animation and repeat that 3 times.
Proposed changes
In the
SectionContent
component, we are adding a new optionalid
prop.In the
SectionContent
component, we are tracking url params and looking fortarget
.If the value of the target param matches the passed value in the
id
prop, we'll temporarily add a blink animation.We are also updating the coming soon section title as per this ticket: https://jira.newfold.com/projects/PRESS1/issues/PRESS1-161
We still need to push an update to the coming soon / ecomm modules to update the referrer link to include the URL params.
Demo
Bluehost-Plugin.-.mp4
Type of Change
Checklist
Further comments