[dolphin] [Bug 518285] Location bar doesn't render correctly in the edit mode with non-Breeze application styles

[Méven]

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

Méven <me...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|dad191bb92022631a2bba5675c9 |https://invent.kde.org/syst
                   |a0aad2b1df5d8               |em/dolphin/-/commit/76f6eb0
                   |                            |7739290bd7e7a61946f51d7329f
                   |                            |b75545
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from Méven <me...@kde.org> ---
Git commit 76f6eb07739290bd7e7a61946f51d7329fb75545 by Méven Car, on behalf of
Filip Fila.
Committed on 04/06/2026 at 08:52.
Pushed by meven into branch 'master'.

DolphinNavigatorsWidgetAction: Let non-Breeze QStyles style the non-toolbar
navbar how they want

Currently when Dolphin's location bar is moved from the main toolbar, it stops
being a nice `FrameLineEdit` that QStyles can style as they please.

That's because the current code is only adjusted for Breeze, which wants to
fill the entire navbar frame with a plain rectangle.

Unfortunately, non-Breeze styles suffer from bad looks because this Breeze
design is applied for them too.

In order to fix this, this MR restores the styling power back to QStyles by
removing the rectangle, all the while keeping it for Breeze.

| Before (Breeze) | After (Breeze) |
| ------ | ------ |
|![DolphinURL-before_Breeze_](/uploads/b526076352023862e2f17cef744802a0/DolphinURL-before_Breeze_.png){width=698
height=134}        | 
![DolphinURL-after_Breeze_](/uploads/eb60d5c2a6dff129c4d200fef5f8c5ab/DolphinURL-after_Breeze_.png){width=715
height=138}      |

| Before (Oxygen) | After (Oxygen) |
| ------ | ------ |
| 
![DolphinURL-before_Oxygen_](/uploads/2d8f19debbd5410b8f0882ed20b34f83/DolphinURL-before_Oxygen_.png){width=714
height=114}      |
![DolphinURL-after_Oxygen_](/uploads/fc6a1bed3de647534494951a7b113d5e/DolphinURL-after_Oxygen_.png){width=687
height=129}       |

| Before (MSWindows9x) | After (MSWindows9x) |
| ------ | ------ |
|  
![DolphinURL-before_MSWindows9x_](/uploads/88ebd20a2be5d9052165ea417517b4a3/DolphinURL-before_MSWindows9x_.png){width=715
height=107}     | 
![DolphinURL-after_MSWindows9x_](/uploads/575e7a944238113714c70b7705b3854a/DolphinURL-after_MSWindows9x_.png){width=702
height=100}      |

| Before (Darkly) | After (Darkly) |
| ------ | ------ |
| 
![DolphinURL-before_Darkly_](/uploads/76f28b8175498b6a611953c05554c4e8/DolphinURL-before_Darkly_.png){width=675
height=123}      | 
![DolphinURL-after_Darkly_](/uploads/7b9b15eaab10ee7731d55c60af7591bf/DolphinURL-after_Darkly_.png){width=705
height=144}      |

| Before (Fusion) | After (Fusion) |
| ------ | ------ |
| 
![DolphinURL-before_Fusion_](/uploads/de27a5028986d797407bfe66b2e9916b/DolphinURL-before_Fusion_.png){width=701
height=118}      |  
![DolphinURL-after_Fusion_](/uploads/7f5c7acd1c433fcba8b96e402cf12a21/DolphinURL-after_Fusion_.png){width=692
height=122}     |


| Before (Kvantum) | After (Kvantum) |
| ------ | ------ |
|
![DolphinURL-before_Kvantum](/uploads/21c2126044ba31c3bc00610e34a71ec1/DolphinURL-before_Kvantum.png){width=697
height=135}       |  
![DolphinURL-after_Kvantum](/uploads/ea5287e7ba59e1f9b449c1b002d3b634/DolphinURL-after_Kvantum.png){width=700
height=123}     |

FIXED-IN:

M  +6    -3    src/dolphinnavigatorswidgetaction.cpp

https://invent.kde.org/system/dolphin/-/commit/76f6eb07739290bd7e7a61946f51d7329fb75545

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