Fix Browser issues - Githubissues

Koniverse / SubWallet-Mobile

SubWallet is the comprehensive non-custodial wallet solution for Polkadot, Substrate & Ethereum ecosystems

Apache License 2.0

35 stars 33 forks source link

Fix Browser issues #873

Closed saltict closed 1 year ago

saltict commented 1 year ago

@Sokol142196 please move all remaining issue of browser to this issue

[x] Show incorrect dApp on Favorites list in case add favorites from Recent list Steps:
1. Go to the Browser tab
2. Access to connect.subwallet.app
3. Access to another dApp
4. Access to connect.subwallet.app again
5. On the Recent list, click on "Add favorite" for the connect.subwallet.app
6. Go to the Favorites list then observe the connect.subwallet.app
Actual: the favorite status of the "connect.subwallet.app" = OFF https://share.icloud.com/photos/05d-Z_mkm1yjjJ4VvDLCLBr3Q
[x] Show empty page when click on item that is not dApp in the favorites list Steps:
1. Go to the Browser tab
2. Click on search box then input "subwallet"
3. Visit to "search Duckduckgo"
4. Back to Browser home
5. Go to the Recent list and add the recent (in step 3) to favorite
6. Back to Browser home
7. On the Favorite, click on the item above
Actual: Show empty page
- Do not show items on the Favorites list https://share.icloud.com/photos/0654J6XQG2LXzf52ew-nN7qGQ
[x] Improve performance in case search
[x] Allow user search by dApp name and dApp URL
[x] Show incorrect connection status after connecting dApp successfully Steps:
1. Go to the Browser tab
2. Visit to "alephzero.artzero.io"
3. Connect dApp successfully
4. Observe the connection status
Actual: Connection status = Disconnected https://share.icloud.com/photos/0bc3jULnVM0m7IfjYx964TACw

Expect: Connection status = Connected
[x] Update background of the web-view
[x] UI bug confirmation screen

[x] Recent list
- [x] Improve: Do not save search criteria if the criteria is not dApp
- [x] Improve: Still save the recent multi-time although user access to the old recent Expect: Only 1 item when consecutive dapp access

ThaoNguyen998 commented 1 year ago

Version test: https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-197-878-merge-2023-07-12T03-53-38.ipa Devices test: IP 11 Validation passed.