-
Notifications
You must be signed in to change notification settings - Fork 26
[1164] Fun facts section #1207
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
base: main
Are you sure you want to change the base?
[1164] Fun facts section #1207
Conversation
elvbom
left a comment
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.
Great work! Some comments on the code below
|
Another though - i'd suggest order of landing page to be:
I'm thinking we want whats relevant to users first, and imo thats them looking at the data. Whats your thoughts? Also - I really like the about us but wonder if it should be a slider/similar, cause right now users get stuck-ish at scroll, and i think its something thats only relevant first time you see it and then you wanna be able to skip easily. Thoughts? |
|
PS. aware that scroll steps are out of scope of this PR! :) |
Great feedback @elvbom! There is a skip button for the animated scroll section, so users don't need to see it. Maybe it's too hidden? 🤔 Was initially thinking like panorama so it can be a bit of an intro if a user is new to the site? Can definitely try re-ordering though to see what we like better 😄 |
✨ What’s Changed?
Added a basic fun fact section to the new landing page component.
📸 Screenshots (if applicable)
Desktop:


Mobile:

📋 Checklist
🛠 Related Issue
Closes #1163