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

[SDK-1556] Apply window height style to root document for Passwordless UI #1878

Merged
merged 1 commit into from
Jun 5, 2020

Conversation

stevehobbsdev
Copy link
Contributor

@stevehobbsdev stevehobbsdev commented Jun 5, 2020

Changes

Determines the window height using JavaScript and applies height to document root when using the Passwordless UI. This takes into account any window chrome provided by mobile devices.

This is a patch that was already applied to the primary Lock UI, but not passwordless.

Before
image

After
image

References

Fixes #1826

Testing

Tested manually using Browserstack + local iOS 11 Pro Simulator.

  • This change adds unit test coverage
  • This change adds integration test coverage
  • This change has been tested on the latest version of the platform/language

Checklist

@stevehobbsdev stevehobbsdev added this to the vNext milestone Jun 5, 2020
@stevehobbsdev stevehobbsdev requested a review from a team June 5, 2020 13:31
@stevehobbsdev stevehobbsdev changed the title Apply window height style to root document [SDK-1556] Apply window height style to root document Jun 5, 2020
@stevehobbsdev stevehobbsdev changed the title [SDK-1556] Apply window height style to root document [SDK-1556] Apply window height style to root document for Passwordless UI Jun 5, 2020
@stevehobbsdev stevehobbsdev merged commit 2321761 into master Jun 5, 2020
@stevehobbsdev stevehobbsdev deleted the fix/passwordless-height branch June 5, 2020 13:39
jfromaniello pushed a commit to jfromaniello/auth0-lock that referenced this pull request Jul 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Passwordless sign-in hidden on Safari
2 participants