Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the information sharing screen usable on tiny screens. #4658

Merged
merged 1 commit into from
Jun 7, 2018

Conversation

mandrigin
Copy link
Contributor

@mandrigin mandrigin commented Jun 6, 2018

e.g. an iPhone app on an iPad
This doesn't make it pretty, just usable.

Steps to test:

  • Open Status iOS on an iPad
  • Expect to be able to pass the information sharing screen

status: ready

@mandrigin mandrigin requested review from jeluard and flexsurfer June 6, 2018 14:36
Copy link
Contributor

@jeluard jeluard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't that conflict with #4655 ?

@mandrigin
Copy link
Contributor Author

@jeluard nope. restricting our app to iPhone-only won't really restring it from being installed launched on an iPad. In that case, it will appear in a tiny window in the centre of the screen. This window has exactly iPhone 4 dimensions. I tested our UI there and this was the only unusable screen I found, so I fixed that :)

@mandrigin
Copy link
Contributor Author

before
simulator screen shot - ipad pro 9 7-inch - 2018-06-06 at 15 55 02

after
simulator screen shot - ipad pro 9 7-inch - 2018-06-06 at 17 41 15

@jeluard
Copy link
Contributor

jeluard commented Jun 6, 2018

@mandrigin Right but we don't intend to support iPad anyway.
I guess it's not an intrusive change so doesn't really matter.

@mandrigin mandrigin self-assigned this Jun 6, 2018
@status-comment-bot
Copy link

status-comment-bot commented Jun 6, 2018

@statustestbot
Copy link

72% of end-end tests have passed

Total executed tests: 29
Failed tests: 8
Passed tests: 21

Failed tests (8)

Click to expand
1. test_send_eth_from_wallet_sign_now

Tap on SignTransactionButton
Looking for GotItButton

E selenium.common.exceptions.NoSuchElementException: Message: 'GotItButton' is not found on screen, using: 'accessibility id:got-it-button'

Device sessions:

2. test_send_eth_to_request_from_wallet

Wait for GotItButton
Looking for GotItButton

E selenium.common.exceptions.NoSuchElementException: Message: 'GotItButton' is not found on screen, using: 'accessibility id:got-it-button'

Device sessions:

3. test_delete_1_1_chat

Wait for GotItButton
Looking for GotItButton

E selenium.common.exceptions.NoSuchElementException: Message: 'GotItButton' is not found on screen, using: 'accessibility id:got-it-button'

Device sessions:

4. test_wallet_error_messages

Tap on SignTransactionButton
Looking for full text: 'Wrong password'

E Failed: Wrong password error did not appear

Device sessions:

5. test_transaction_send_command_one_to_one_chat

Wait for GotItButton
Looking for GotItButton

E selenium.common.exceptions.NoSuchElementException: Message: 'GotItButton' is not found on screen, using: 'accessibility id:got-it-button'

Device sessions:

6. test_transaction_send_command_wrong_password

Tap on SignTransactionButton
Looking for full text: 'Wrong password'

E selenium.common.exceptions.TimeoutException: Message: 'BaseElement' is not found on screen, using: 'xpath://*[@text="Wrong password"]', during '20' seconds

Device sessions:

7. test_send_eth_to_request_in_one_to_one_chat

Wait for GotItButton
Looking for GotItButton

E selenium.common.exceptions.NoSuchElementException: Message: 'GotItButton' is not found on screen, using: 'accessibility id:got-it-button'

Device sessions:

8. test_send_stt_from_wallet_via_enter_recipient_address

Tap on SignTransactionButton
Looking for GotItButton

E selenium.common.exceptions.NoSuchElementException: Message: 'GotItButton' is not found on screen, using: 'accessibility id:got-it-button'

Device sessions:

Passed tests (21)

Click to expand
1. test_sign_transaction_twice
Device sessions:

2. test_swipe_and_delete_1_1_chat
Device sessions:

3. test_one_to_one_chat_messages
Device sessions:

4. test_copy_and_paste_messages
Device sessions:

5. test_send_transaction_from_daap
Device sessions:

6. test_faucet_console_command
Device sessions:

7. test_username_and_profile_picture_in_chats
Device sessions:

8. test_open_transaction_on_etherscan
Device sessions:

9. test_incorrect_password
Device sessions:

10. test_backup_seed_phrase_and_recover_account
Device sessions:

11. test_offline_messaging_1_1_chat
Device sessions:

12. test_contact_profile_view
Device sessions:

13. test_public_chat
Device sessions:

14. test_public_chat_management
Device sessions:

15. test_network_switch
Device sessions:

16. test_browse_link_entering_url_in_dapp_view
Device sessions:

17. test_profile_picture
Device sessions:

18. test_switch_users
Device sessions:

19. test_set_up_wallet
Device sessions:

20. test_qr_code_and_its_value
Device sessions:

21. test_sign_message_from_daap
Device sessions:

@lukaszfryc lukaszfryc self-assigned this Jun 7, 2018
@lukaszfryc
Copy link
Contributor

I've checked it on Android Emulator with WVGA (800x480) screen. It looks not bad. No buttons are covered:

zrzut ekranu 2018-06-07 o 09 24 44

zrzut ekranu 2018-06-07 o 09 25 16

zrzut ekranu 2018-06-07 o 09 25 27

zrzut ekranu 2018-06-07 o 09 40 04

zrzut ekranu 2018-06-07 o 09 43 29

zrzut ekranu 2018-06-07 o 09 44 48

@lukaszfryc lukaszfryc mentioned this pull request Jun 7, 2018
e.g. an iPhone app on an iPad
This doesn't make it *pretty*, just usable.
@rasom rasom force-pushed the bugfix/make-sharing-screen-usable-on-iPhone-4 branch from 9416447 to f49cfc0 Compare June 7, 2018 09:12
@rasom rasom merged commit f49cfc0 into develop Jun 7, 2018
@rasom rasom deleted the bugfix/make-sharing-screen-usable-on-iPhone-4 branch June 7, 2018 09:12
@chadyj
Copy link
Contributor

chadyj commented Jun 7, 2018

Great changes to make things usable on an iPad!

#4662 will make it optimized :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

8 participants