Skip to content

GUI: always show 'Close dialog on finish' option #5751

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

landam
Copy link
Member

@landam landam commented May 23, 2025

Currently an option to close the dialog

image

is shown only for tools which are producing some output data. It's seems to be like a bug. This option should be shown always. Compare eg. r.compress (no output) with r.buffer (raster output).

@landam landam self-assigned this May 23, 2025
@landam landam added bug Something isn't working GUI wxGUI related Python Related code is in Python labels May 23, 2025
@landam landam added this to the 8.5.0 milestone May 23, 2025
@landam landam marked this pull request as draft May 23, 2025 15:45
@landam landam marked this pull request as ready for review May 23, 2025 15:47
@petrasovaa
Copy link
Contributor

This needs to be done in a different way, otherwise it's going to close dialogs that have text output that a user would want to inspect.

@petrasovaa petrasovaa closed this May 23, 2025
@landam landam deleted the forms_close_finish branch May 23, 2025 19:42
@landam landam restored the forms_close_finish branch May 25, 2025 14:29
@landam
Copy link
Member Author

landam commented May 25, 2025

@petrasovaa I agree with you. On the other hand, I think the checkbox Close dialog on finish should be shown for all tools. Current default value is false in the settings, so the dialogs are not closed automatically. Currently the checkbox is not shown for tools like r.compress and others which is wrong. We could hide the checkbox for tools which have file output defined like v.db.select, so tools for which we expect that they print something important to stdout. But it will not probably true for 100% tools. I think it's still better to show the checkbox for all tools. Current behaviour may be confusing for some user. For some tools the checkbox is shown, for some not.

@landam landam reopened this May 25, 2025
@landam landam requested a review from petrasovaa May 25, 2025 14:29
@landam landam removed the bug Something isn't working label May 25, 2025
@lindakarlovska
Copy link
Contributor

@petrasovaa I agree with you. On the other hand, I think the checkbox Close dialog on finish should be shown for all tools. Current default value is false in the settings, so the dialogs are not closed automatically. Currently the checkbox is not shown for tools like r.compress and others which is wrong. We could hide the checkbox for tools which have file output defined like v.db.select, so tools for which we expect that they print something important to stdout. But it will not probably true for 100% tools. I think it's still better to show the checkbox for all tools. Current behaviour may be confusing for some user. For some tools the checkbox is shown, for some not.

I agree with showing this option for all cases of modules. Another thing is (which is not a part of this PR) which option should be checked on default - I can imagine having it as global settings - which means that when a user once checks Close dialog on finish, the settings will be applied globally so that other dialogs launched later will be automatically closed.

@lindakarlovska lindakarlovska self-requested a review May 27, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI wxGUI related Python Related code is in Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants