-
Notifications
You must be signed in to change notification settings - Fork 13
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: landing & config page UX improvements #235
feat: landing & config page UX improvements #235
Conversation
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.
self review.. i'm going to try to provide a config-provider so we can call reset/save/etc there and remove the localStorage items
7bea1db
to
f9187b2
Compare
- moved About to the bottom and made it more friendly - adjusted colors to improve contrast - header to match other web utiities we have
f9187b2
to
e24a80c
Compare
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.
Thanks! Content lgtm, but it felt a bit distracting, so played a bit and ended up moving it to bottom.
Pushed a commit with some additional changes:
- moved About to the bottom and made it more friendly for someone who wants to learn how things work
- adjusted colors to improve contrast and indicate when button is enabled
- tweaked the header to include (beta) and match style of other web utiities we have like https://explore.ipld.io/ or https://cid.ipfs.tech/
@SgtPooki @2color If these look good, feel free to merge or rephrase if anything sounds off.
Preview
@lidel that looks a lot cleaner. i'll fix up the CI and get this merged. |
Title
feat: landing & config page UX improvements
Description
Fixes #58
Summary of changes
TODO: need to fix this so it doesn't require a refresh, which will not work on first-hit for subdomainthis todo is done.)Landing page view
Config page view
Notes & open questions
FYI that inputs are still JSON based for now.
Change checklist