-
-
Notifications
You must be signed in to change notification settings - Fork 333
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
[website] Use product demo video of the onboarding on the landing page #3555
Conversation
Certainly! Looks great! |
That's cool, makes the product look a lot more "human" somehow :D Edit: personally I don't like that there's no timeline unless in full screen mode - you don't know how long the video will last right away, plus can't skip forward if you want to - not sure if that can be fixed, but probably it can. |
closes #3557
Agreed, I added the native video controls. not sure why we tried to roll something custom here. |
@apedroferreira I have merged this and we are good for a release! Thanks. |
Comparing the experience on:
Overall it felt that 3554 (before) had the worst tradeoff. It feels like there are two clear ways to make the previous experience better. So 3555 looks like a clear step forward, it brings context, nice 👍 |
Something we could consider
Preview: https://deploy-preview-3555--mui-toolpad-docs.netlify.app/toolpad/#hero-container-right-area