QR code overlaps GUI on Trezor T display

[bosomt]

software: Monero GUI 0.16.0.0
hardware: Trezor model T fw version 2.3.1
OS: windows 10 , latest version

How to reproduce:

1. go to receive tam
2. press Show on device
3. device freezes, application freezes

[xiphon]

@ph4r05

[rating89us]

I'm using Trezor FW 2.3.0 in Monero GUI 0.16.0.0 and I can tap on the buttons below the QR code, but it is really difficult because they are covered.
If I remember correctly, in previous Trezor FW versions you could tap on the QR code to make it disappear.

[ph4r05]

@rating89us I've just tested it and had similar results. Due to the limited space, it is more problematic to hit the button but I succeeded. So I hope it is not a blocker issue, but it is not good for UX for sure.

I don't think we can scale down the QR code much because it could become unreadable. QR codes for monero addresses are bigger compared to the currencies the QR code was deployed for (e.g., BTC).

So the question is how it should be handled properly...

[rating89us]

It would be better if the QR code were moved down and the bottom buttons hidden while QR code is shown. This would give enough space for the path on top to be displayed.

Tapping or swipping the QR code should make it disappear.

Like this:

[ph4r05]

@rating89us hmm I like the idea that swipe/tap removes the QR code!

[rating89us]

I also noticed that the screen of @bosomt's device is a little larger than the screen of my Trezor Model T. Maybe older versions of Model T had a different screen with lower touch sensitivity on the borders.

[tsusanka]

@rating89us @ph4r05 have you ever used this feature? I mean the fact that Trezor is capable of showing the QR code. Because I have not and it seems to me as a feature for 0.01% users. So I am actually wondering if we should not remove the QR code completely. What is your take on that?

[tsusanka]

Fixed on our side in firmware, trezor/trezor-firmware@0f9a245. The fix will be most likely included in the next firmware 2.3.2 to be released on August 5th.

Feel free to close this issue.

[selsta]

@tsusanka Thank you for the update, closing.