https://bugs.kde.org/show_bug.cgi?id=522650

[email protected] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
                 CC|                            |alessandro.chitarrini51@gma
                   |                            |il.com
             Status|REPORTED                    |CONFIRMED

--- Comment #5 from [email protected] ---
I think I found the cause, and it is not theme-specific.

In src/dolphinnavigatorswidgetaction.cpp the Empty Trash button is a flat
QPushButton sharing a QHBoxLayout with the url navigator. It stays hidden until
you enter trash:/, and a hidden widget adds nothing to the layout height. Once
it shows up the row becomes as tall as the button, which is taller than the
navigator, so the toolbar grows and everything below it moves down.

I measured it by capturing the same window in the home folder and in trash:/
and comparing the two images: the Places sidebar moves down by exactly 2 px,
and the separator under the toolbar goes from y=45 to y=47. No existing toolbar
widget changes height, they are just re-centred in a taller row.

Letting the navigator decide the row height fixes it:

   
emptyTrashButton->setSizePolicy(emptyTrashButton->sizePolicy().horizontalPolicy(),
QSizePolicy::Ignored);

Same measurement with that change: sidebar dy = 0, separator stays at y=45.

The amount is the size hint difference between QPushButton and KUrlNavigator in
the active style, so it can be 0 with some styles. That would explain why it
was not reproducible in comment #2.

The Add Network Folder and Open KDE Connect buttons in the same layout are
built the same way, so they are probably affected too, but I could not test
that as neither knetattach nor kdeconnect is installed here.

Verified against 26.08.1, since master currently needs a newer KIO than the one
available here (KIO/RenameFileWarningDialog is missing in KIO 6.30.0). The code
at the patch site is identical in both.

Tested on Arch Linux, Qt 6.11.2, KIO 6.30.0, Breeze via qt6ct, Hyprland on
Wayland with no Plasma installed. Merge request coming.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to