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

Minimum window size (1002 pixels) is too large for smaller screens (800 pixels). #297

Closed
dreirund opened this issue Jul 12, 2024 · 4 comments
Milestone

Comments

@dreirund
Copy link

I use Merkaartor release version 0.19.0 on Artix GNU/Linux, installed from the Arch Linux repositories.

The minimum window size (on my system) is 736x1002 pixels, which in height is more than my laptop's display resolution (1280x800).

I cannot resize the window to fit on my screen.

So I hereby request to support also smaller screens, ideally VGA (640x480), or, if really not possible, 800x600 I think is the minimum that should be supported nowadays, to also suit small portable Linux devices.

Here is the output of xwininfo for the merkaartor window resized to smallest possible size:

xwininfo: Window id: 0x5200011 "untitled - Merkaartor v0.19.0-64bit"

  Absolute upper-left X:  134
  Absolute upper-left Y:  -37
  Relative upper-left X:  1
  Relative upper-left Y:  11
  Width: 736
  Height: 1002
  Depth: 24
  Visual: 0x415
  Visual Class: TrueColor
  Border width: 0
  Class: InputOutput
  Colormap: 0x5200005 (installed)
  Bit Gravity State: NorthWestGravity
  Window Gravity State: NorthWestGravity
  Backing Store State: NotUseful
  Save Under State: no
  Map State: IsViewable
  Override Redirect State: no
  Corners:  +134+-37  -410+-37  -410--165  +134--165
  -geometry 736x1002+134+-37

Regards!

@Krakonos
Copy link
Member

Krakonos commented Sep 8, 2024

This is caused by the status bar. The minimum window width depends on the text length there. Hiding viewport coordinates here would fix the issue, not sure if it's acceptable? It could be made into a setting. There are a couple of other misc items that can be made optional.

@dreirund
Copy link
Author

dreirund commented Sep 8, 2024

This is caused by the status bar. The minimum window width depends on the text length there.

This is unexpected to the user.

The status bar text should then be truncated (and indicated with "…" that it is been truncated), with optional scroll arrows or a mouseover that shows the whole status bar in multiline.

Status bar content should not restrict window shrinkage.

@Krakonos
Copy link
Member

Krakonos commented Sep 9, 2024

Agreed. Unfortunately, the QStatusBar component isn't too flexible. I did find a way to somewhat shrink the window. I can now get 490x618 or so (depends on the exact coordinates, fonts etc.), I find this sufficient. Check PR #303 if it works for your use-case. Also if you don't mind me asking, what device are you trying to run on? Most modern devices have more resolution.

image

@Krakonos Krakonos added this to the 0.20.1 milestone Sep 17, 2024
@Krakonos
Copy link
Member

Krakonos commented Oct 2, 2024

Merged to master, will be release in 0.20.1

@Krakonos Krakonos closed this as completed Oct 2, 2024
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

No branches or pull requests

2 participants