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

Input method height #1011

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

maltee1
Copy link
Contributor

@maltee1 maltee1 commented Mar 22, 2022

This time, uncontroversial changes only

@deepbluev7
Copy link
Member

Yep, those look fine to me, but does it actually work on the PP?

@maltee1
Copy link
Contributor Author

maltee1 commented Mar 23, 2022

It works now. But it may not work everywhere. Do you want to get someone with e.g. Phosh to test it or just push it and wait if someone complains?

Comment on lines +28 to +34
Component.onCompleted: {
fullHeight = height
}
onHeightChanged: {
if(!Qt.inputMethod.visible)
fullHeight = height
}
Copy link
Member

Choose a reason for hiding this comment

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

Won't this break on phone rotations? :D

Copy link
Contributor Author

@maltee1 maltee1 Mar 23, 2022

Choose a reason for hiding this comment

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

are you saying that when rotating the phone with the keyboard up, fullHeight won't change? Or fullHeight will change when it shouldn't?

Disregarding the fact that Nheko is more or less unusable in landscape mode (at 200% scaling, with On-screen keyboard), the former is not an issue, because the keyboard is stowed on rotation. The latter does happen, but I think that's actually desirable.

There's also the Screen QML Type which provides a property desktopAvailableHeight and height. At least the latter will be persistent and could be used. But that would require us to somehow determine what kind of platform we're on, I imagine people on Desktop want their images to resize according to the window size (what about multiple screens?). Another settings toggle should be the last option if nothing else is deemed satisfactory :D

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