-
Notifications
You must be signed in to change notification settings - Fork 8
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: show connected wallet info #22
base: main
Are you sure you want to change the base?
Feat: show connected wallet info #22
Conversation
@DimaKush is attempting to deploy a commit to the BuidlGuidl Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for the PR, @DimaKush! I appreciate your detailed description and screenshot, very professional. Let's take a step back and consider the goal of the issue. The idea is to give the user visual feedback for whether they are (1) on the Thinking about state for a moment, there are several possible paths:
How might you represent each of these states in a way that helps the user discover the next step they need to take? Right now you show green icons for |
@DimaKush i think you did a good job. but i have an idea for something. if you want you can let me do it or make improvement to yours or do it instead @derrekcoleman if @DimaKush is okay with it, i can do this. or we could just go which whatever he has. this is just the idea i had for it and i thought to share |
@FrankiePower Yep looks good, but on mobile we don't have a lot of space at Header. So I moved this to main page. |
okay @DimaKush . also if you don't mind i'd like to see a preview (screenshot) of what it looks like |
I'm not working on this but reading through your work I thought we could mix it together. Since I would prefer the current user status information to be displayed in the header, it would be nice to have a more concise unique badge, instead of two, that is responsible for displaying the user status. Enregistrement.de.l.ecran.2025-01-10.a.15.59.12.movWhat do you think? |
i think i also like what @All-Khwarizmi is proposing. we can find a way to implement the responsiveness for other screens. but then you'd also have to find a way to display what @DimaKush is trying to display. |
I love this conversation! This design "problem" appears all of the time - we want the information to be obvious/available (e.g. in the header of all pages), but need it to be concise/unobtrusive (e.g. make sense on mobile). Personally, I like the pattern of using attention-grabbing icons (imagine ❗ for error states) with on-hover/on-click tooltips that the user can easily discover to learn more in 1-2 sentences. |
Move Status Component to HeaderMoved batch status from homepage to header for better UX. Users can now see their allowlist/check-in status from any page. Before
After
Changes
Screenshots |
Description
Added builder status indicators (batch membership and check-in status) to the wallet connection dropdown.
Changes
Testing
Screenshot
Related Issues
Closes #7
This PR includes:
The changes are focused and minimal, adding the requested functionality without unnecessary modifications.
ENS/address: 0x845D62b5405aAAe6b95CF3fbf33968A1E71a9cAD