-
Notifications
You must be signed in to change notification settings - Fork 19
Move homepage & appstore UIs to main repo #285
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
Conversation
There's definitely some weirdness with the current setup, and you may wish to prevent merge until I write some docs (maybe with @willbach to cover things I don't yet know) about the particulars of modifying it. Or, just add that as a TODO and merge if it looks OK, as the UI is in need of some love anyway. |
@dr-frmr @nick1udwig I think the present state is a good spot to cut a PR - the homepage hasn't been Vite-ified yet but there is enough to test and double-check in the Register flow that I would prefer to close this one out before moving on. |
TODOs as of now:
|
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.
"LGTM"
Also, fix scrollbar issues.
To test: