search

trezor / trezor-firmware

:lock: Trezor Firmware Monorepo
https://trezor.io
Other
1.35k stars 652 forks source link

Recovery: Backspace button is too small #288

Closed tsusanka closed 5 years ago

tsusanka commented 5 years ago

Many consider the backspace button way too small and difficult to press.

Reported by @andrewkozlik

mmahut commented 5 years ago

I second this, while in recovery I wanted to hit the backspace button while confirmed the bad word. #286 would fix this for my use case too, therefore I think one of these should make it in.

tsusanka commented 5 years ago

I think this looks quite alright. We should consider doing this for BIP-39 as well, I'll check out what is the widest word there.

emu00000001

emu00000000 emu00000002

(downtown is one of the widest words)

prusnak commented 5 years ago

Can you please try the word "remember"?

tsusanka commented 5 years ago

The fifth of November! And also mountain, which is quite large.

emu00000000 emu00000001

prusnak commented 5 years ago

Looks great! Let's investigate this for BIP39 as you suggest.

andrewkozlik commented 5 years ago

FYI, the widest word using the Trezor normal font is "downtown" (93 pixels) for SLIP-0039 and "mushroom" (95 pixels) for BIP-0039. But now I realized that we are using the monospace font for the word, So it makes no difference :-D. All eight-letter words are 96 pixels.

tsusanka commented 5 years ago

For BIP-39:

xemu00000000 xemu00000001 xemu00000002 xemu00000003

It's all good.

tsusanka commented 5 years ago

I'm also considering moving it one more pixel to the left (but you can barely see it):

emu00000000

tsusanka commented 5 years ago

PR in https://github.com/trezor/trezor-firmware/pull/302