Diffusion Krita b216c98e98b1

Ensure that checkboxes are shown in menus

Authored by huftis on Aug 4 2019, 4:04 PM.

Description

Ensure that checkboxes are shown in menus

The old code tried to add an icon for all menu items, even for
empty icon names. Most themes (including the default ‘Fusion’ theme,
but not KDE’s ‘Breeze’) don’t display checkboxes when a menu item
already has an icon. The result was that unchecked checkboxes
were invisible and checked checkboxes looked strange (like large,
empty rectangles).

With this commit, icons are only added for actions that actually
*have* icons, and the checkboxes are thus correctly displayed no matter
what theme the user has.

Details

Committed
huftisAug 13 2019, 8:14 PM
Parents
R37:56c77bb7f071: Fix mixed Zoom, Pan, Rotate on macOS
Branches
Unknown
Tags
Unknown