Skip to content
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: add mobile header with connected info #140

Merged
merged 5 commits into from
May 23, 2024

Conversation

philbow61
Copy link
Contributor

@philbow61 philbow61 commented May 14, 2024

Description:

This PR makes some stylistic changes to the mobile header as well as adding the mobile wallet info.

Fixes #121

Copy link

vercel bot commented May 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
governance-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2024 3:20pm

@denviljclarke denviljclarke changed the base branch from develop to test May 16, 2024 10:15
@denviljclarke denviljclarke changed the base branch from test to develop May 16, 2024 10:15
@denviljclarke
Copy link
Contributor

Looks good! I just added some min-height to the balance display component and added an alert to notify users when they've successfully copied their address. Good stuff 🫡

Copy link
Contributor

@denviljclarke denviljclarke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🫡

@denviljclarke denviljclarke requested review from a team, chapati23 and bowd and removed request for a team May 16, 2024 11:06
Copy link
Contributor

@chapati23 chapati23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@philbow61 philbow61 merged commit c53e9e0 into develop May 23, 2024
3 checks passed
@philbow61 philbow61 deleted the fix/mobile-sidebar-missing-wallet-information branch May 23, 2024 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants