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

Add ReCaptcha screen #6135

Merged
merged 2 commits into from
May 11, 2022
Merged

Add ReCaptcha screen #6135

merged 2 commits into from
May 11, 2022

Conversation

pixlwave
Copy link
Member

@pixlwave pixlwave commented May 9, 2022

This PR adds the views, but it isn't yet part of the flow and the coordinator only has an initial implementation so far. Additionally this PR adds support for dark mode to the MXKAuthenticationReCaptchaWebView.

Note: Voiceover only partially works on the ReCaptcha form and Dynamic Type doesn't seem to work at all. There is also potentially more work required to get the sizing of the form to be optimal on a larger range of devices.

Part of issue #5650

Fixes #6054

Light Begin Light Challenge Dark Begin Dark Challenge
Screenshot 2022-05-09 at 4 43 42 pm Screenshot 2022-05-09 at 4 43 54 pm Screenshot 2022-05-09 at 4 44 10 pm Screenshot 2022-05-09 at 4 44 24 pm
MXK Begin Before MXK Challenge Before MXK Begin After MXK Challenge After
IMG_0388 IMG_0389 IMG_0386 IMG_0387

@pixlwave pixlwave changed the base branch from develop to doug/5649_auth_email_verification May 9, 2022 16:04
@pixlwave pixlwave marked this pull request as ready for review May 9, 2022 16:15
@pixlwave pixlwave requested review from a team, SBiOSoftWhare, ismailgulek and aringenbach and removed request for a team and ismailgulek May 9, 2022 16:15
@github-actions
Copy link

github-actions bot commented May 9, 2022

📱 Scan the QR code below to install the build for this PR.
🔒 This build is for internal testing purpose. Only devices listed in the ad-hoc provisioning profile can install Element Alpha.

QR code

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/H3rJAN

Copy link
Contributor

@SBiOSoftWhare SBiOSoftWhare left a comment

Choose a reason for hiding this comment

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

Good to see @MainActor, Task and await coming

Copy link
Contributor

@aringenbach aringenbach left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from doug/5649_auth_email_verification to develop May 10, 2022 13:09
Support dark mode in MXKAuthenticationRecaptchaWebView.
Begin implementing the ReCaptcha coordinator.
@pixlwave pixlwave merged commit d0494cf into develop May 11, 2022
@pixlwave pixlwave deleted the doug/5650_auth_captcha branch May 11, 2022 08:57
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.

reCAPTCHA is not dark-theme compatible
3 participants