trezor / trezor-suite

Trezor Suite Monorepo
https://trezor.io/trezor-suite
Other
720 stars 251 forks source link

Warning banners alternating in odd priority #14514

Open Hermez-cz opened 1 month ago

Hermez-cz commented 1 month ago

Describe the bug If there are two warning banners, only one is shown in certain priority. For example if I have a banner that my device is not backed up (pretty serious) and I enable Polygon, the backup banner is covered by the messaging system banner related to the Matic - Pol migration. Messaging banners have priority, they might be pretty urgent in some cases. But it shouldn't ideally cover another potentially urgent banner about my device not being backed up.

Info:

How to reproduce Steps to reproduce the behavior:

  1. Wipe the Suite app data (if you've already dismissed the Polygon migration banner).
  2. Use a device without a backup, with only Bitcoin enabled, and observe the red "Backup" warning modal.
  3. Enable Polygon, and notice the blue informative migration banner covering the red warning.
  4. Disable Polygon, and see the blue banner disappear, revealing the red warning again.
  5. Repeat as needed.

Expected behavior Banners would be visible both at once.

Screenshots https://github.com/user-attachments/assets/e23e3c53-5d4f-46c9-b40c-f39edf65dcfd

MiroslavProchazka commented 3 weeks ago

See more here