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
chore: GRO-213: Adding LazyLoadComponent to FooterBanner #7076
Conversation
@The-Beez-Kneez - can you link to the JIRA ticket in the description? |
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.
Cool!!
<ScrollIntoView | ||
selector="#download-app-banner" | ||
behavior="smooth" | ||
> |
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 remembered this after pairing -- whenever we want something to jump to part of the page, we can wrap it in this component, and if we want to to animate we can use the behavior='smooth'
prop
#mergeOnGreen |
Ticket: GRO-213
There were performance optimizations recommended by Damon for the footer.
The image in the footer should be lazy loaded and be a real
img
tag to increase accessibility.New layout: