-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat: Teaser video #491
feat: Teaser video #491
Conversation
Branch preview✅ Deployed successfully in branch deployment: |
8ab0f94
to
526876b
Compare
526876b
to
8bd2a7a
Compare
8796a2e
to
741efd2
Compare
0e056a7
to
1bb6ea3
Compare
{ready ? ( | ||
<div className={`${css.imageWrapper} ${showButton ? css.visible : ''}`}> | ||
<ButtonBase target="_blank" rel="noreferrer" href={ALPHA_TELEGRAM_LINK} className={css.button}> | ||
{isMobile ? <Logo className={`${css.logo} ${!showLogo ? css.hidden : css.show}`} /> : null} |
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.
We can use the mobileVideo
class here instead of isMobile
but I don't think you can remove complete reliance on the hook so I'll leave it up to you.
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.
This code that was using the viewport width hook was removed.
What it solves
Teaser landing page