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

Rework GUI dialogs: Add manager, and scrollbars when necessary #138

Open
we7u opened this issue Jun 4, 2019 · 10 comments · Fixed by #166
Open

Rework GUI dialogs: Add manager, and scrollbars when necessary #138

we7u opened this issue Jun 4, 2019 · 10 comments · Fixed by #166

Comments

@we7u
Copy link
Member

we7u commented Jun 4, 2019

Dialogs were added/extended from the original code which used hard-coded connections beween the window frame and the widgets. There was no included window manager and most dialogs are also missing scrollbars. Dialogs were forced to fixed sizes so resizing wouldn't mess up the widget stackup. This made it difficult to use on small screen sizes as portions of the dialogs would be off-screen and with no scrollbar people couldn't get to parts of the dialog.

Change dialogs to include a manager widget / add scrollbars where needed / make them resizeable.

@we7u we7u self-assigned this Jun 4, 2019
@tvrusso
Copy link
Member

tvrusso commented Jun 4, 2019

Is this issue minor enough that you know how to fix it for the next release? If so, might want to add it to the ToDo column of the 2.2.0 kanban.

@we7u
Copy link
Member Author

we7u commented Jun 4, 2019

There are quite a few dialogs so I don't know quite when they'll all be done. Might take me a while.

@tvrusso
Copy link
Member

tvrusso commented Jun 4, 2019

Well, at this point we've been releasing less often than once a year. Your call whether to commit to it as a 2.2.0 feature. If you decide on it, add it to the KanBan, and you can always remove it if it ain't gonna happen by next year.

@tvrusso tvrusso added this to In progress in Release 2.2.0 Feb 23, 2020
@we7u
Copy link
Member Author

we7u commented Feb 24, 2020

First tweaks pushed as branch "dialogs". More to do yet.

@we7u we7u mentioned this issue Feb 25, 2020
@tvrusso tvrusso linked a pull request Feb 26, 2020 that will close this issue
Release 2.2.0 automation moved this from In progress to Done Feb 27, 2020
@godfreja
Copy link
Contributor

I believe Curt still has work to do on this and this should be reopened. Curt can close this if I am wrong.

@godfreja godfreja reopened this Feb 27, 2020
Release 2.2.0 automation moved this from Done to In progress Feb 27, 2020
@tvrusso
Copy link
Member

tvrusso commented Feb 27, 2020

Yes, indeed, this was mistakenly closed when I foolishly added it as a "related issue" to the PR, thinking that when Github said "Merging this PR may close these issues" it meant "may" not "will."

I will not do that again.

@we7u
Copy link
Member Author

we7u commented Feb 28, 2020

I have code running that checks screen size and dialog size, opening the dialog up to the smaller of the two (Thanks to Jason Godfrey for the idea!). If it finds a smaller screen size it also opens the dialog at 0,0 (upper left corner of the screen). So far this seems to work well. I'm continuing work on those features plus modifying the rest of the dialogs. This should be ready in the next few weeks for a pull request, sooner if I can manage it.

@we7u
Copy link
Member Author

we7u commented Mar 4, 2020

The latest code (yet to be pushed to my Xastir fork) looks at the screen size when creating a dialog.

If dialog is larger than the screen size, it opens the dialog up at size screen-size-minus-decorations, activates scrollbars, and locates it at 0,0 (upper left). I hard-coded numbers of "6" and "27" to compensate for the left/right and top/bottom decoration sizes, but that depends on the window manager selected, its settings, and the font used.

If dialog is smaller than screen size, open it like normal and just set max dialog size. It can be made smaller by the operator at which point scrollbars will appear.

I also hard-coded a "+10" where I set max dialog sizes: This appears to be the size of the added frame just inside the decorations but outside the form. I have more dialogs to convert but will push changes (to my fork) soon so others can test.

I haven't found a general way to compute decoration sizes on the system after trying quite a few methods. It looks like hard-coding decoration sizes is our only option. I'm interested in any comment about that. Perhaps we can later create variables for those, settable on some configuration dialog?

@tvrusso tvrusso moved this from In progress to To do in Release 2.2.0 Oct 17, 2021
@tvrusso
Copy link
Member

tvrusso commented Oct 17, 2021

It's been over a year since this issue was last updated, so I've moved it back out of "in progress" and into the "to do" column.

Curt, if you consider it complete (or even "as much as I'm gonna do") then let's close it.

@tvrusso tvrusso removed this from To do in Release 2.2.0 Jul 27, 2023
@tvrusso
Copy link
Member

tvrusso commented May 28, 2024

I'm gonna go ahead and remove this from the Xastir 2.2.2 project. Progress was made on the effort, but it's been 4 years since any work was done on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants