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 issues on mobile/touch devices #329

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

alex-spataru
Copy link

Some fixes for mobile/touch devices. Changes tested on both Android 4.2.2 and Windows 10.
More details about the changes are available on the description of each commit.

Touch devices do not interact very well with the "hover" property of the mouse area. This causes the widgets that use this object to get "stuck" with the color used when the item is clicked, even if the user is not touching/clicking the widget.

To fix this, we simply disable the hover feature when running on a mobile device.
We show the application maximized on a mobile device, and we ensure it resizes to screen size by setting size to (0,0)
On desktop devices, the behavior of the application will be exactly as the "previous" behavior.

Tested on Android 4.2.2 and Windows 10
@oKcerG
Copy link
Contributor

oKcerG commented Nov 9, 2015

First commit is equivalent to an earlier PR : #322
I have no opinion of the second commit except that I didn't experienced the bug, maybe create an issue for this one ?

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