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 keyboard height issue #483

Merged
merged 1 commit into from
Jan 10, 2016
Merged

Fix keyboard height issue #483

merged 1 commit into from
Jan 10, 2016

Conversation

dylanjeffers
Copy link
Contributor

Use the print lines to debug. It seems that mActivity.getWindow()... changes rctx.top to a non-zero number, which causes the app to permanently shift upwards . To replicate this error, make an app that calls the keyboard. The hotfix I added was to set rctx.top to zero.

Temporarily forcing Rect().top to always be 0.
@akshayaurora
Copy link
Member

where is the issue you are trying to solve? detail the version of kivy and py4a being used in the issue.

@inclement
Copy link
Member

@akshayaurora As i remember, @SapienTech was finding that the keyboard code permanently offset the kivy window. I didn't look into the issue, but does the solution seem reasonable to you? I'm not sure if forcing top to 0 is correct on all devices.

@dylanjeffers
Copy link
Contributor Author

@akshayaurora @inclement kivy is v1.9.1-dev , Buildozer is 0.31dev with the py4a it pulls by default. I agree with inclement that I'm not sure if this fix will work on all devices, but I have not heard any issues from the users of my app.

@ghost
Copy link

ghost commented Jan 10, 2016

I can confirm that #544 can be solved with #483 merge. The fix by SapienTech is working very well, if the app is fullscreen or if the statusbar is shown.

akshayaurora added a commit that referenced this pull request Jan 10, 2016
@akshayaurora akshayaurora merged commit 49e287e into kivy:master Jan 10, 2016
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