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

print: use hierarchical menus for styles #18186

Merged
merged 2 commits into from
Jan 12, 2025

Conversation

ralfbrown
Copy link
Collaborator

Nobody noticed that the print view has an option to add a style to the image(s) being printed, which is now as unmanageable as the export module's was just after adding the camera specific styles. This PR upgrades the print settings the same way the export module was, with a hierarchical set of menus for the styles and a preview of the style's effect on hover.

@ralfbrown ralfbrown added the scope: UI user interface and interactions label Jan 10, 2025
@TurboGit TurboGit added this to the 5.0.1 milestone Jan 11, 2025
@TurboGit
Copy link
Member

I use the print module but didn't notice as since a long time I have presets that select the proper style.

@TurboGit TurboGit added bugfix pull request fixing a bug priority: high core features are broken and not usable at all, software crashes labels Jan 11, 2025
Copy link
Member

@TurboGit TurboGit left a comment

Choose a reason for hiding this comment

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

This breaks the print setting preset menu. Click on the hamburger top-right and darktable crashes.

src/libs/print_settings.c Outdated Show resolved Hide resolved
@ralfbrown
Copy link
Collaborator Author

I don't get a crash from the hamburger menu, though I didn't test that until after fixing the set/reset bits.

Copy link
Member

@TurboGit TurboGit left a comment

Choose a reason for hiding this comment

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

Works for me now, thanks!

@TurboGit TurboGit merged commit 5d3836e into darktable-org:master Jan 12, 2025
6 checks passed
TurboGit pushed a commit that referenced this pull request Jan 12, 2025
* print: use hierarchical menus for styles
@TurboGit
Copy link
Member

Merged in master and 5.0.x branch.

@ralfbrown
Copy link
Collaborator Author

Release note:

  • harmonized style selection in print settings with the update made in the export module for 5.0.0.

@ralfbrown ralfbrown deleted the print_styles branch January 12, 2025 06:46
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.

2 participants