BeamMW / ios-wallet

Beam iOS Wallet
https://beam.mw/downloads
Apache License 2.0
21 stars 9 forks source link

Rework nodes statuses #531

Closed dariatarakanova closed 2 years ago

dariatarakanova commented 3 years ago

1) Remove word "online" 2) Add "lock" icon to mobile and own node. 3) leave "circle" icon for random node. 4) Add "arrow" icon that leads to Node sittings image

5) Mobile node

5.1) Connected https://zpl.io/beGl6gm image

5.2) Updating https://zpl.io/a8qX5Px image

5.3) Cannot connect https://zpl.io/adQl6pK image

6) Own node

6.1) Connected https://zpl.io/ag5lRQM image

6.2) Cannot connect https://zpl.io/V147RQE image

7) Random node

7.1) Connected https://zpl.io/bJdXMg3 image

7.2) Reconnecting to another random node https://zpl.io/amxGjwm image

7.3) Cannot connect https://zpl.io/aBAX4Dq image

sinaizd commented 2 years ago

this item needs to be finalized (text).In the case when the user performs a repeat scan or the mobile node is sync with the network, the phrase "faster connection" can be misleading. By going to the "node" screen, the user will choose a different connection that will disrupt synchronization.

5.2) Updating https://zpl.io/a8qX5Px image

dariatarakanova commented 2 years ago

@Denis-Asipenko - point 5.2 was updated ( in task's body) according to our discussion last week.

AndreyDyakonov commented 2 years ago

own node:

  1. When wallet cannot connect to own node - the entire message is not fully displayed in master: photo_2021-10-11_12-44-16 in design: image

  2. When wallet connect to own node - status displayed incorrectly in master: photo_2021-10-11_12-44-20 in design: image

dariatarakanova commented 2 years ago

@DenisDemyanko please make the following changes in text for connected nodes (green status) @Denis-Asipenko FYI

1) "online: mobile node" should be just "online" 2) "online: own node xxx.xxx.xxx:xxxx" should be just "online" 3) "online: random node supports online transactions only" leave as it is

image

AndreyDyakonov commented 2 years ago

The entire text should be visible

in master:

in design:

AndreyDyakonov commented 2 years ago

Checked