Xapp Status Icon Menu Items - very inconsistent paddings.
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
Research direction
Reproduce the inconsistent horizontal padding in the XApp status-icon menus using the listed icons and GTK Inspector, then locate the xapp status-icon menu implementation. Compare menus with and without checkboxes across themes; done means absent checkbox space is no longer reserved while checkbox-bearing menus retain correct padding.
Written by the indexing model from the issue text.
Description
Cinnamon 4.6.6
xapp-common 1.8.8
Icons in use
Cjs-Message: 07:52:26.839: JS LOG: [LookingGlass/info] Adding XAppStatusIcon: blueberry (org.x.StatusIcon.PID-2033-5)
Cjs-Message: 07:52:26.842: JS LOG: [LookingGlass/info] Adding XAppStatusIcon: redshift-gtk (org.x.StatusIcon.PID-1766-5)
Cjs-Message: 07:52:26.844: JS LOG: [LookingGlass/info] Adding XAppStatusIcon: keepassxc (org.x.StatusIcon.PID-3808-12)
Cjs-Message: 07:52:26.846: JS LOG: [LookingGlass/info] Adding XAppStatusIcon: hp-systray (org.x.StatusIcon.PID-3808-13)
Cjs-Message: 07:52:26.847: JS LOG: [LookingGlass/info] Adding XAppStatusIcon: flameshot (org.x.StatusIcon.PID-3808-14)
Cjs-Message: 07:52:26.849: JS LOG: [LookingGlass/info] Adding XAppStatusIcon: mintUpdate.py (org.x.StatusIcon.PID-2380-5)
Cjs-Message: 07:52:26.851: JS LOG: [LookingGlass/info] Adding XAppStatusIcon: nm-applet (org.x.StatusIcon.PID-1794-5)
Cinnamon warning: Log level 128: posix_spawn avoided (automatic reaping requested) (fd close requested)
Cjs-Message: 07:52:26.884: JS LOG: [LookingGlass/info] Hiding XAppStatusIcon (we have an applet): nm-applet (org.x.StatusIcon.PID-1794-5)
Some of the menus displayed for these various icons have anomalous horizontal padding consistently across a variety of different GTK themes. Examining the menus with GTK inspector I think there's space being left for an absent checkbox. The only one that looks correct is redshift which has checkboxes. The mintupdate menu doesn't appear to have the absent checkbox issue.
E.g.
- Dominant language
- C
- Stars
- 169
- Forks
- 49
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from linuxmint/xapp
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
-
Difficulty 3/5 1-2 days Newbie friendliness 72/100
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
enhancement
Difficulty 1/5 Under an hour Newbie friendliness 92/100
-
enhancement
Difficulty 1/5 Under an hour Newbie friendliness 94/100
-
feature request
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100