Bug 503932 - Menubar missing icons and misalignment
Summary: Menubar missing icons and misalignment
Status: REPORTED
Alias: None
Product: partitionmanager
Classification: Applications
Component: general (other bugs)
Version First Reported In: Git
Platform: Neon Linux
: LO wishlist
Target Milestone: ---
Assignee: Andrius Štikonas
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-05-08 14:52 UTC by Tamás Králl
Modified: 2025-05-16 20:09 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
Missing icons in Device section (21.31 KB, image/png)
2025-05-08 14:53 UTC, Tamás Králl
Details
Missing icons in Partition section (19.21 KB, image/png)
2025-05-08 14:53 UTC, Tamás Králl
Details
Missing icon in Tools section (9.33 KB, image/png)
2025-05-08 14:54 UTC, Tamás Králl
Details
Misalignment and missing icon in Settings section (23.51 KB, image/png)
2025-05-08 14:54 UTC, Tamás Králl
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tamás Králl 2025-05-08 14:52:25 UTC
SUMMARY
In the menubar the Device, Partition, Tools, and Settings sections have missing icons. The Settings section also has a bit of weird misalignment.

STEPS TO REPRODUCE
1. Open KDE Partition Manager
2. Navigate to one of Device, Partition, Tools, or Settings sections in the menubar 

OBSERVED RESULT
Missing icons and misalignment

EXPECTED RESULT
Appropriate icons and no random misalignment

SOFTWARE/OS VERSIONS
KDE Partition Manager: 25.07.70
KDE Frameworks: 6.15.0
Qt: Using 6.9.0 and built against 6.9.0
KDE neon Unstable Edition (Wayland)
Build ABI: x86_64-little_endian-lp64
Kernel: linux 6.11.0-25-generic
Comment 1 Tamás Králl 2025-05-08 14:53:14 UTC
Created attachment 181072 [details]
Missing icons in Device section
Comment 2 Tamás Králl 2025-05-08 14:53:40 UTC
Created attachment 181073 [details]
Missing icons in Partition section
Comment 3 Tamás Králl 2025-05-08 14:54:04 UTC
Created attachment 181074 [details]
Missing icon in Tools section
Comment 4 Tamás Králl 2025-05-08 14:54:34 UTC
Created attachment 181077 [details]
Misalignment and missing icon in Settings section
Comment 5 Andrius Štikonas 2025-05-08 15:06:38 UTC
We never had icons there, I guess because it's hard to pick a suitable icon for those actions... Perhaps we could add mount icon though to the same one as Dolphin uses.

There is no misalignment in the settings menu though. Checkboxes and (missing) icons are not meant to be aligned (theoretically you could have both checkbox and an icon).
Comment 6 Tamás Králl 2025-05-08 15:38:57 UTC
If I understand correctly, here's what causes the "misalignment":
1. The space between the checkbox and its text is for a (currently not used) icon
2. the text in the bottom half of the Settings section is aligned to those at the top
3. therefore an empty space is created before the text in the bottom half of the Settings section

I feel like this could be improved. Right now the the Settings section (dialog? context menu?) looks a bit odd with the empty spaces.
Comment 7 Andrius Štikonas 2025-05-08 16:00:09 UTC
(In reply to hottog from comment #6)
> I feel like this could be improved. Right now the the Settings section
> (dialog? context menu?) looks a bit odd with the empty spaces.

I don't think partition manager has any control over that. It probably comes from Qt. So besides adding some icons, to hide the issue, there isn't anything I  can do regarding alignment.
Comment 8 Tamás Králl 2025-05-08 16:14:36 UTC
I see. I think adding a few icons (if there are available icons for these actions) would go a long way. And perhaps if the two checkboxes were next to each other, maybe that would make the Settings menu look a bit tidier.

Anyway, thanks for the quick responses and for this application!
Comment 9 Andrius Štikonas 2025-05-09 01:29:28 UTC
Git commit ef7b4081aee82965f2dba6778373153d3e64c541 by Andrius Štikonas.
Committed on 09/05/2025 at 01:28.
Pushed by stikonas into branch 'master'.

Re-order settings menu.

M  +1    -2    src/gui/partitionmanagerui.rc

https://invent.kde.org/system/partitionmanager/-/commit/ef7b4081aee82965f2dba6778373153d3e64c541
Comment 10 Tamás Králl 2025-05-15 09:39:10 UTC
I've noticed that KTorrent has the same kind of settings menu as what partition manager had. Should I open a new bug/wishlist report for that as well?
Comment 11 Tamás Králl 2025-05-15 10:11:02 UTC
Dolphin has a similar settings entry as the "Panels/Toolbars Shown" entry in Partition Manager/KTorrent, and it uses the "view-sidetree-symbolic" icon for it. So perhaps that icon could be used in Partition Manager/KTorrent as well.

And other possible icons where there's currently no icons in Partition Manager:
- "drive-symbolic" for Device/SMART Status
- "edit-symbolic" for Partition/Edit Mount Point
- "media-mount-symbolic" for Partition/Mount
- "info-symbolic" for Tools/File System Support

I know these are generic icons but in my opinion it's better to have a generic icon than no icon at all.
Comment 12 Andrius Štikonas 2025-05-15 18:31:20 UTC
(In reply to hottog from comment #10)
> Should I open a new bug/wishlist report for that as well?

Well, you have an example commit of reordering menu items in partitionmanager. Instead of opening wishlist report, just open a merge request :)
Comment 13 Tamás Králl 2025-05-16 08:10:36 UTC
(In reply to Andrius Štikonas from comment #12)
> Well, you have an example commit of reordering menu items in
> partitionmanager. Instead of opening wishlist report, just open a merge
> request :)

Could you point me to the file I need to edit? I've created a KDE Identity account and checked the ktorrent repository, but I have trouble understanding how it all works (I've never done any software development before).

But if I know where to look I'll probably be able to figure out what I need to do :)
Comment 14 Andrius Štikonas 2025-05-16 18:39:48 UTC
(In reply to hottog from comment #13)
> (In reply to Andrius Štikonas from comment #12)
> > Well, you have an example commit of reordering menu items in
> > partitionmanager. Instead of opening wishlist report, just open a merge
> > request :)
> 
> Could you point me to the file I need to edit? I've created a KDE Identity
> account and checked the ktorrent repository, but I have trouble
> understanding how it all works (I've never done any software development
> before).
> 
> But if I know where to look I'll probably be able to figure out what I need
> to do :)

Hmm, I briefly looked at it now, and it is somewhat confusing, I don't fully understand it...

Anyway, it seems that these files are relevant:

https://invent.kde.org/network/ktorrent/-/blob/master/ktorrent/gui.cpp?ref_type=heads#L373
(here the icon for "Show statusbar" is being set, but I guess it's an invalid icon name...)

(some of the menu structure is also in .rc files, e.g. in https://invent.kde.org/network/ktorrent/-/blob/master/ktorrent/ktorrentui.rc?ref_type=heads)
Comment 15 Andrius Štikonas 2025-05-16 18:40:20 UTC
Anyway, probably best to have a separate ticket there, it's another application...
Comment 16 Tamás Králl 2025-05-16 20:09:36 UTC
I think I'll start with some easier code contributions (this c++ code is quite difficult to understand just by looking at it) and I'll let this one be done by someone with more experince than me. Tomorrow I'll create a proper bug report for KTorrent as you recommended.

I'll keep this bug report open because the missing icons part is still vaild.