-
Notifications
You must be signed in to change notification settings - Fork 30
feat: Console UI Enhancements #387
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?
Conversation
Website preview 🔗✨ |
This looks amazing 🚀 Great improvement to the UI |
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 on the new design, I really like it!
I noticed some inconsistency with button styling, some areas still use the old rounded pattern while others have the new boxed design. Since there’s currently no centralized Button component, this would be a great opportunity to implement one.
Going the extra mile, you could also add multi-theme support with theme-aware components instead of updating individual styles across all files. (that's just an idea)
Can we do something about the red logo on the dark background - doesn't look great. Otherwise I really like the change! |
d488adc
to
228a7a7
Compare
Thanks for reviewig it @BravoNatalie - it is ready for another round. |
I've increased the contrast and made the gradient a bit lighter. |
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.
I can definitely be convinced to ship this and update the copy buttons later, but it would be nice to make the copy UX consistent now
</label> | ||
<button | ||
onClick={(e) => { | ||
e.stopPropagation() |
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.
suggestion: consider creating a new component for this copy button if one doesn't exist already - here's an example with better UX (ie, visual indicator that the copy succeeded): https://github.com/storacha/bluesky-backup-webapp-server/blob/main/src/components/CopyButton.tsx
e.preventDefault() | ||
e.stopPropagation() | ||
try { | ||
await navigator.clipboard.writeText(did) |
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.
yea we're doing this in multiple places now - I'd really like to DRY this out and make the copy UX consistent
Console UI Enhancements Proposal
Web View

Mobile View

Iframe View
