In file applets/kicker/package/contents/config/main.xml an option 'showIconsRootLevel' is mentioned. It has following description:
"Whether to show icons on the root level of the menu."
This change provides implementation for this option.
In file applets/kicker/package/contents/config/main.xml an option 'showIconsRootLevel' is mentioned. It has following description:
"Whether to show icons on the root level of the menu."
This change provides implementation for this option.
Manual test:
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
Attaching screenshot with current look of menu (or with disabled option) and new look with enabled option and also a configuration widget.
Kicker intentionally doesn't have icons on the root level to not conflict with the favorites column and reduce visual noise.
Favorite icons are much larger, it's hard to confuse them with root level menu icons. If it's deemed as informational noise, it's disabled by default. But in case it's not, it may be enabled if this change is applied. And in my opinion it looks more consistent in style with 'modern menu' since 'modern menu' has those icons enabled.
Edit: and it looks more consistent with menu editor which show those root level icons. It looks weird for me when menu editor shows those icons but menu does not.
Can you please update this patch? It doesn't apply anymore. It would also be nice if you used Arcanist/arc in the future: https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist_to_post_patches
I'm still a bit grumpy about this and dragging my feet! My gut still says it's both option clutter and UI noise :-)
However:
It's a go.
I don't think I have access for that. Also I don't have Arc at the moment, and there's no indication in web interface that I have capability to do it.