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

Android: Improper size of window and crash with measure tool #787

Closed
krticka opened this issue Oct 30, 2016 · 1 comment · Fixed by #1296
Closed

Android: Improper size of window and crash with measure tool #787

krticka opened this issue Oct 30, 2016 · 1 comment · Fixed by #1296
Assignees
Milestone

Comments

@krticka
Copy link
Contributor

krticka commented Oct 30, 2016

Steps to reproduce

  1. Try to measure length of object
  2. Result is located in very small window, not readable
  3. Move window to upper dock position and try to move window to different position.

Actual behaviour

Result of measuring is not readable, window is too small. Resize of window does not work. After move of window to upper dock position window is enlarges automatically, result is readable, but application accidentally crashes after move of window back above map field.
screenshot_2016-10-30-09-21-50 1

Expected behaviour

Readable result of measurement, resize of window is working, no crash on window move.

Configuration

Mapper Version: 0.6.6
Operating System: Android 6.0.1 (Samsung Galaxy Note 4)

@dg0yt
Copy link
Member

dg0yt commented Oct 30, 2016

It is a little bit more complicated, and probably outside of our source code.

  • The measure widget is a "dock widget". It is initially "floating" (undocked), but it gets docked and resized when you move it to the top (or any other border). (I even contributed a change to Qt to have a titlebar at all, for closing and moving.) There shouldn't be a crash, though.
  • The initial size should get some review.
  • Basically, resizing does work as well, as it does on the desktop.
  • One issue especially on Android devices is insufficient high-dpi support in the version Qt we are using at the moment (Qt 5.5.1). The tiny icons in the widget title bar are an outcome of this. Thus I won't look into this issue before I can make relases based on Qt 5.6 (Switch to Qt 5.6 #673). (I think I can release my local work in the next weeks.)

This issue reminds us of the fact that the Android port of Mapper is still very much a proof-of-concept.

@dg0yt dg0yt added this to the v0.7.0 milestone Nov 5, 2016
@dg0yt dg0yt modified the milestones: Next minor release, v0.7.0 Apr 12, 2017
@dg0yt dg0yt self-assigned this Jul 20, 2019
@dg0yt dg0yt modified the milestones: Next minor release, v0.9.0 Dec 5, 2019
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 a pull request may close this issue.

2 participants