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

Fix Missing Hint Text on Android - Move the hint text inside the capture div #10339

Merged
merged 2 commits into from
Mar 29, 2024

Conversation

charleyf
Copy link
Contributor

@charleyf charleyf commented Mar 29, 2024

🛠 Summary of changes

This PR fixes an issue where the hint text was not showing on Android/Chrome by moving the hint text inside the div that the Acuant SDK binds to. Before this PR the hint text would not show up on Android/Chrome most of the time.

I've tested that the hint text appears with this PR for:

  • Android/Chrome
  • iOS/Chrome
  • iOS/Safari

Copy link
Contributor

@night-jellyfish night-jellyfish left a comment

Choose a reason for hiding this comment

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

Tested this out and I see hint text! 🎉 Great catch!

@charleyf
Copy link
Contributor Author

I do want to note that I think @amirbey implemented it this way before and I think I changed it in the course of other work at some point. So thanks for getting it right originally Amir 😅

Copy link
Contributor

@eileen-nava eileen-nava left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this. 🙏🏻 I tested it on Android/Chrome and it worked well.

@charleyf charleyf merged commit 92135c6 into main Mar 29, 2024
2 checks passed
@charleyf charleyf deleted the charley/show-hint-text-on-android-chrome branch March 29, 2024 17:43
@aduth aduth mentioned this pull request Apr 2, 2024
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.

3 participants