Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Separate visual for security key PIN when using NFC key #2129

Closed
tildelowengrimm opened this issue Dec 17, 2019 · 12 comments · Fixed by #3192
Closed

Separate visual for security key PIN when using NFC key #2129

tildelowengrimm opened this issue Dec 17, 2019 · 12 comments · Fixed by #3192

Comments

@tildelowengrimm
Copy link

tildelowengrimm commented Dec 17, 2019

330DBC1E-3C37-4D9E-B0B7-73053E807A1B

Currently, we use an image which looks like a YubiKey 5Ci on the PIN entry screen. I would be better to have a more generic NFC key visual when using an NFC key rather than a lightning one.

@karenkliu
Copy link

How's this?

nfc key mock

assets.zip

@tildelowengrimm
Copy link
Author

The icon on the security key looks very much like a WiFi or RSS icon rather than an NFC visual?

@karenkliu
Copy link

@tomlowenthal Please send an example NFC visual

@tildelowengrimm
Copy link
Author

tildelowengrimm commented Jan 29, 2020

nfc_cms7f5ef84210

@karenkliu
Copy link

How's this?
nfc key mock

@karenkliu
Copy link

bump @tomlowenthal

@tildelowengrimm
Copy link
Author

LG2M

@karenkliu
Copy link

here are assets.zip

@karenkliu karenkliu removed their assignment Jul 2, 2020
@karenkliu
Copy link

@jumde Hey is there any movement on this issue? It's just been hanging out on the design board - are we still doing this?

@karenkliu
Copy link

@anthonypkeane What's happening with this issue?

@jumde
Copy link
Contributor

jumde commented Jan 12, 2021

@karenkliu @anthonypkeane - I can wrap this up post 1.23.1 release.

@kjozwiak
Copy link
Member

Verification PASSED on iPhone 11 running iOS 14.4 using the following build:

1.24 (21.3.25.18)
5Ci Lighting NFC
IMG_3617 IMG_3618

Verification PASSED on iPhone 6s+ running iOS 12.4.1 using the following build:

1.24 (21.3.25.18)

IMG_0158


Verification PASSED on iPad Air (3rd Gen) running iOS 14.4 using the following build:

1.24 (21.3.25.18)

IMG_0251


Verification PASSED on iPad Mini 4 running iOS 13.7 using the following build:

1.24 (21.3.25.18)

IMG_0496

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants