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

New Profile Modal #2317

Closed
JimmyMow opened this issue May 30, 2019 · 1 comment

Comments

2 participants
@JimmyMow
Copy link
Member

commented May 30, 2019

Right now for a user to access their public key (and host), lndconnect info, and on-chain address they click this QR code to the left of the balance:

Screen Shot 2019-05-30 at 10 13 50 AM

There are a few UX issues here. One of them is the fact that node information (Profile information) shouldn't live where the current on-chain address lives, and it is confusing to a user on how to find such information.

Screen Shot 2019-05-30 at 10 13 56 AM

I propose we create a Profile link under Settings in our new navigation.

Screen Shot 2019-05-30 at 10 16 13 AM

When clicking on this new Profile link, we will open a modal similar to the Settings modal:

Screen Shot 2019-05-30 at 10 17 05 AM

The left nav will contain items like Node information (Pubkey) and lndconnect.

This allows all the profile type node info to live in its own area within the application, and moves us closer to removing that weird QRCode next to the balance 😄

@JimmyMow JimmyMow added this to the v0.5.0-beta milestone May 30, 2019

@mrfelton mrfelton referenced this issue Jun 10, 2019

Merged

feat(ui): Implement new profile pages #2392

5 of 6 tasks complete

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 11, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 11, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 11, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 11, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 11, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 12, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 12, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 12, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 12, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Jun 12, 2019

@mrfelton

This comment has been minimized.

Copy link
Member

commented Jun 12, 2019

Closed via #2392

@mrfelton mrfelton closed this Jun 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.