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

Keycard Sign up UI issues #21675

Closed
mariia-skrypnyk opened this issue Nov 25, 2024 · 7 comments · Fixed by #21938
Closed

Keycard Sign up UI issues #21675

mariia-skrypnyk opened this issue Nov 25, 2024 · 7 comments · Fixed by #21938
Assignees
Labels
keycard mobile-core ui-polish Aesthetics; not a functional bug UX

Comments

@mariia-skrypnyk
Copy link

mariia-skrypnyk commented Nov 25, 2024

Bug Report

Problem

Keycard Sign up was implemented before design changed so now some UI should be fixed according to updated design version here:
https://www.figma.com/design/YGm3igIOAcwMqUVJWCJ6f1/Keycard?node-id=3387-32189&node-type=frame&t=SW9vLzJIZqyYhTaq-0

Reproduction

  1. Install app
  2. Prepare empty Keycard
  3. Go to Create Profile > Use Keycard
  4. Scan Keycard
  5. Pass all other steps needed to Create Profile using a Keycard

ISSUE 1: [Android] Background should become darker when NFT scanner opens

Design VS Implementation

Image

ISSUE 2:

Design VS Implementation

Image

  • different image
  • add info icon

ISSUE 3:

Design VS Implementation

Image

  • Keycard is empty (is should be added in all screens where it was missing)
  • black status bar (Android)
  • custom color of Let's go button
  • link on upvote

ISSUE 4:

Design VS Implementation

Image

  • "the" Keycard

ISSUE 5:

Additional Information

  • Status version: nightly
  • Operating System: Android, iOS
@flexsurfer
Copy link
Member

ISSUE 1: [Android] Background should become darker when NFT scanner opens
this should be a separate issue, because NFC scan dialog has temporary implementation, it should be properly implemented

@flexsurfer flexsurfer added the UX label Jan 7, 2025
@Parveshdhull Parveshdhull self-assigned this Jan 14, 2025
@Parveshdhull
Copy link
Member

Thank you @mariia-skrypnyk for creating this issue.

add info icon

Do we have content ready for this info icon?

Also please can you share link to original figma while where this feedback/review is done?

@Parveshdhull
Copy link
Member

Parveshdhull commented Jan 16, 2025

Summary

  • Issue 1 - should be fixed separately
  • Issue 2 - Partially fixed - Info button content remaining
  • Issue 3 - Fixing some issues.
    • Probably design review is outdated, as per figma title should be "Keycard is empty"
    • Not updating bottom info message, as we are working on feature implementation instead.
  • Issue 4 - As per figma, we should keep "This isn't a keycard"

@mariia-skrypnyk
Copy link
Author

Hey @Parveshdhull !

Should I report Issue 1 separately?

Do we have content ready for this info icon?

Also please can you share link to original figma while where this feedback/review is done?

No as I see on design.
Only button for now.
Asked design team here.
https://www.figma.com/design/YGm3igIOAcwMqUVJWCJ6f1/Keycard?node-id=4337-45746&m=dev

@mariia-skrypnyk
Copy link
Author

mariia-skrypnyk commented Jan 17, 2025

  1. Sorry, I missed bottom name in Issue 2.
    Can you change it in current PR?
Image

https://www.figma.com/design/YGm3igIOAcwMqUVJWCJ6f1/Keycard?node-id=3-89231&p=f&m=dev

Issue 3 - Fixing some issues.
Probably design review is outdated, as per figma title should be "Keycard is empty"

you are right, design has changed!

Not updating bottom info message, as we are working on feature #21932 > instead.

ok, thanks!

  1. Issue 4

Issue 4 - As per figma, we should keep "This isn't a keycard"

also change of the design, thanks.

@Parveshdhull
Copy link
Member

Asked design team here.

Thank you. I have added info button with not implemented flag. We can enable it once button content is available.

Should I report Issue 1 separately?

Yes, thanks.

I missed bottom name in Issue 2.

Updated the PR with change.

@Parveshdhull
Copy link
Member

Parveshdhull commented Jan 20, 2025

@mariia-skrypnyk Please feel free to update this issue or open new for leftovers. (Issue 1 and info button content)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keycard mobile-core ui-polish Aesthetics; not a functional bug UX
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants