Skip to content

Fix guides menu toggle - #22084

Merged
TurboGit merged 1 commit into
darktable-org:masterfrom
zisoft:fix-guides-menu-toggle
Aug 31, 2026
Merged

Fix guides menu toggle#22084
TurboGit merged 1 commit into
darktable-org:masterfrom
zisoft:fix-guides-menu-toggle

Conversation

@zisoft

@zisoft zisoft commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

Some iop modules have the "show guides" checkbox (crop, rotate & perspective, flip, lens, clipping...).

The state of this checkbox can also be toggled via the module's presets menu but in that case the checkbox in the module itself is not toggled.

Activating this menu item also throws a GTK critical error:

Gtk-CRITICAL: gtk_toggle_button_set_active: assertion 'GTK_IS_TOGGLE_BUTTON (toggle_button)' failed

Bildschirmfoto 2026-08-30 um 20 31 51

@zisoft
zisoft force-pushed the fix-guides-menu-toggle branch 3 times, most recently from bc24fc5 to 1eb38ec Compare August 30, 2026 18:57
@piratenpanda

Copy link
Copy Markdown
Contributor

Must have missed that. Thanks!

@TurboGit TurboGit added this to the 5.8 milestone Aug 30, 2026
@TurboGit TurboGit added bugfix pull request fixing a bug priority: high core features are broken and not usable at all, software crashes scope: UI user interface and interactions labels Aug 30, 2026
Comment thread RELEASE_NOTES.md
@da-phil

da-phil commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Just out of curiosity, why do we need to have so many iop modules with their own "show guides" checkbox, when we have a central one in the bottom bar in the bottom right?

@TurboGit

TurboGit commented Aug 30, 2026

Copy link
Copy Markdown
Member

Just out of curiosity, why do we need to have so many iop modules with their own "show guides" checkbox, when we have a central one in the bottom bar in the bottom right?

That's a good question. This has been discussed before I was manager of the project. I was not against but always found that it is a bit redundant. The idea was to have the control closer to the mouse when dealing with modules where guides are often used. It is also a way to record if for such or such module a guided should be displayed, but for this we can keep the entry in the module's preference menu.

So maybe we can re-discuss the removal of the check box in module's main UI. If you feel this important please open a new issue to track this discussion. This should not be part of this PR.

@zisoft
zisoft force-pushed the fix-guides-menu-toggle branch from 1eb38ec to c01bca5 Compare August 31, 2026 05:18

@TurboGit TurboGit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@TurboGit
TurboGit merged commit 3e38ba3 into darktable-org:master Aug 31, 2026
6 checks passed
@zisoft
zisoft deleted the fix-guides-menu-toggle branch August 31, 2026 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix pull request fixing a bug priority: high core features are broken and not usable at all, software crashes scope: UI user interface and interactions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants