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

status bar now has correct height to include back link on iphone X #592

Closed
wants to merge 2 commits into from

Conversation

rgowan
Copy link
Contributor

@rgowan rgowan commented Jul 6, 2018

#445

Fixed issue with the overflowing back link in the status bar by updating the version of 'react-native-safe-area-view'.

simulator screen shot - iphone x - 2018-07-06 at 14 36 11

Tester check-list

  • Tested on multiple devices / OS versions (Test on the physical device(s) you have access to, otherwise use BrowserStack):

    • Galaxy S8 (Android 7.0/8.0)
    • Galaxy S7 (Android 6.0)
    • iPhone X (iOS 11.x)
    • iPhone 8 (11.x)
    • iPhone 7 (iOS 10.x)

    Optional:

    • Google Pixel 2
    • Galaxy S8 Plus
    • Galaxy S7 Edge
    • Galaxy S6
    • iPhone 7+
    • iPhone 6

Accessibility Testing (If applicable)

  • Text-to-Voice (Android: Voice Assistant / iOS: Speak Selection)
  • Large Text (=<200%)
  • Landscape Screen orientation

Weak connection testing (If applicable)

  • Airplane mode
  • 2G/3G Network Settings

@rgowan rgowan self-assigned this Jul 6, 2018
Copy link
Contributor

@RGBboy RGBboy left a comment

Choose a reason for hiding this comment

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

Looks good to me. Would be good to get design signoff too before merging.

@rgowan
Copy link
Contributor Author

rgowan commented Jul 9, 2018

I spoke to Jun and Steve on Friday and they said to go with it.

@@ -39,7 +39,7 @@
"react-native-map-link": "^1.0.6",
"react-native-maps": "^0.21.0",
"react-native-permissions": "^1.1.1",
"react-native-safe-area-view": "robbiemccorkell/react-native-safe-area-view",
"react-native-safe-area-view": "^0.9.0",
Copy link
Contributor

@RGBboy RGBboy Jul 9, 2018

Choose a reason for hiding this comment

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

There is one more part that needs removing later in this file on the resolutions property.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolutions removed, pushed up 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants