Skip to content

Conversation

@gr5
Copy link
Collaborator

@gr5 gr5 commented Dec 3, 2025

This will greatly help the app fit on smaller displays or displays where scaling is > 200%.

This was a new issue for Dale when we switched to QT6 but also has been an issue with people whose screen width is < 1280 pixels.

This will help everyone, even people with large displays as you can choose to allocate less space for the right side tool area and give more display area to other things.

I was able to achieve this by simply changing the minimum widths to 250 for all the .ui files that have to fit in that right pane. The gui editor in QT was very helpful by showing what it would look like as I made the tools wider and thinner.

temp_gif2

Fixes #289

@gr5
Copy link
Collaborator Author

gr5 commented Dec 3, 2025

This fixes issue #289

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

🚀 New build available for commit d8acf43
Download installer here

@githubdoe
Copy link
Owner

Wonderful. Good work. I think you can make it even smaller and let the user decide when it gets too small. I just tried 100 and it was still ok.

@gr5
Copy link
Collaborator Author

gr5 commented Dec 3, 2025

100pix is too narrow I think. Looks like this in qt creator. Again, this is "minimum" width setting - not sure if that's what you played with.

image

Also realize you have to worry about all 4 tabs. And already the first tab is a bit tight. This is how it looks for me at 250px (qt6). Note how the "edge zoom" checkbox is cut off a bit and a few other things.

image

@githubdoe
Copy link
Owner

I could not get it that small so I did not do the right setting. However I would love to be able to scrunch it down that small. What value did you change to do that? I tried minimum setting of the surface analysis tools.

@gr5
Copy link
Collaborator Author

gr5 commented Dec 4, 2025

It's the minimum width but you have to do all 4 tabs and the first tab is tricky - look at the ui file changes by clicking on "files changed" near the top of this page and ignore anything that isn't 250 (some other numbers changed for some reason, I guess because I was stretching and shrinking for testing but those numbers don't seem to matter).

@atsju
Copy link
Collaborator

atsju commented Dec 5, 2025

I tested d8acf43. Looks good. However at 125% GUI was not responding very well and I wasn't able to resize as smooth as I wanted. Only 2 possible sizes like it's not analog slidebar. I don't know if you experience same :

Video.Project.mp4

I'm not against changing minimum size to a very small value as long as default size displays OK.

@gr5
Copy link
Collaborator Author

gr5 commented Dec 5, 2025

That weird behavior - it does that in the older version also. You don't have to go to 125%. You just have to make the application width a little too small. I don't know how small exactly. I tried the version on github from this pr and set to 125% at full screen I was fine but when I shrunk the width of the app a bit it does the same thing yours does.

I don't know how to prevent that weirdness. But basically it happens when there isn't enough room horizontally in the main window of the application. I don't know what sets this.

I think a better test is to not touch that right-side-size-adjuster area and instead make the entire app get smaller and larger. If you do that you can better see where the minimum size comes to play.

I think it's fine as is.

@gr5
Copy link
Collaborator Author

gr5 commented Dec 5, 2025

Dale, where are we with this? I made it about half the minimum width and I could go much smaller but it gets less usable pretty quickly. Can I stick with 250 as the minimum?

@githubdoe
Copy link
Owner

I have never found all the settings to try it. Mine stays stuck at a larger size. I don't have enough time. Is there code I could try? I have master cloned and working.

@gr5
Copy link
Collaborator Author

gr5 commented Dec 5, 2025

7 posts above this one there is a link. Just download that, install it, and try it out.

Or do you want to build it or edit it yourself? The code changes are tiny - basically 4 lines of code but I did it with the ui editor thing. I can show you a quick video of what to change in the ui editor. Would that help? maybe I'll do that.

Also you can download this particular branch if you have the github cli with this command line command:
gh pr checkout 295

But you have to install the github cli and you have to make sure all your edits are saved or commited (if you have any edits since your last big pull request a few weeks ago).

I'll do a video. I might have to do it in a few hours as I think dinner is in 10 minutes.

@gr5
Copy link
Collaborator Author

gr5 commented Dec 6, 2025

Dale I made you a video showing how to try different "minimum widths" for that right side of the app:
https://www.youtube.com/watch?v=7qjd1B8oriI

I tried to keep it extra short (1 minute 37 seconds).

@githubdoe
Copy link
Owner

Thank you for the video. It helped but did not fix my problem. I still can't get the width to too small. I have isolated it to the section called interferogram tools (on mainWindow) by dragging the header of the control so that it is no longer docked. All others also dragged will reduce in size but that one will not even when I have minimum set to 100 and dragged it left on the UI editor. Oh, well. Carry on.

@gr5
Copy link
Collaborator Author

gr5 commented Dec 11, 2025

Dale do you want to approve this so I can merge it? I think Julien is waiting for you to approve it as the code change is of course pretty basic. It's more about is this an improvement to the program or do we need to go farther.

I don't want to go too far as not everyone will be able to find how to use the mouse to make the tools area larger so I want it to shrink to a value that is still usable (just barely usable).

@atsju atsju requested review from atsju and githubdoe December 11, 2025 12:38
Copy link
Collaborator

@atsju atsju left a comment

Choose a reason for hiding this comment

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

Sorry for delay. As explained in comment, I'm OK with this change. Dale is main person of interrest here

@gr5 gr5 merged commit 8ff99d4 into master Dec 12, 2025
14 checks passed
@atsju atsju deleted the issue289 branch December 23, 2025 14:00
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.

UI right bar to adjust for Dale and other screen

4 participants