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

Fixes for positioning instruction view according to text height #126

Merged

Conversation

ezefire
Copy link
Contributor

@ezefire ezefire commented Apr 17, 2020

Fixes made to positioning instruction view using its height when instruction view is below the target.

Fixes made to positioning instruction view correctly in iPad when backgroundView is outer of the right side of the screen.

@quangctkm9207
Copy link
Collaborator

Thank you @ezefire for your PR and time for this project.
I really appreciate that.
I am going to merge it then check if there is no issue, I will release a new version with your change as soon as possible.

@quangctkm9207 quangctkm9207 merged commit 30c1d54 into aromajoin:master Apr 18, 2020
@ezefire
Copy link
Contributor Author

ezefire commented Apr 19, 2020 via email

@quangctkm9207
Copy link
Collaborator

Thank you @ezefire for your kind words and your time.

Best regards,
Quang.

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.

2 participants