Bug 501589

Summary: New address bar: | separators are sometimes thin or bold, also depends on address bar's width
Product: [Frameworks and Libraries] frameworks-kio Reporter: postix <postix>
Component: URL navigatorAssignee: KIO Bugs <kio-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: kdedev, kdelibs-bugs-null, nate
Priority: NOR Keywords: regression
Version First Reported In: 6.12.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 6.14
Sentry Crash Report:
Attachments: Screenshot

Description postix 2025-03-16 16:08:06 UTC
Created attachment 179465 [details]
Screenshot

SUMMARY

When changing the address bar's width, some | separators start to alter between 1 and 2 px width.

See the screenshot from another issue, which shows the | in 2px in one case. The second | is bold.

SOFTWARE/OS VERSIONS
Betriebssystem: Fedora Linux 41
KDE-Plasma-Version: 6.3.3
KDE-Frameworks-Version: 6.12.0
Qt-Version: 6.8.2
Kernel-Version: 6.13.7-200.fc41.x86_64 (64-bit)
Grafik-Plattform: Wayland
Scaling: 1.25
Comment 1 postix 2025-03-16 16:12:51 UTC
I've checked another system with 150% scaling: there are several bold and thin ones for the same path. Looks a bit odd.
Comment 2 TraceyC 2025-03-19 20:30:41 UTC
I see this on Dolphin in git-master as well
Comment 3 Nate Graham 2025-03-21 18:08:09 UTC
Probably a fractional scaling thing.

Regardless, will be fixed by https://invent.kde.org/frameworks/kio/-/merge_requests/1842.
Comment 4 Akseli Lahtinen 2025-04-04 20:01:19 UTC
Git commit 7067680d23229b80dd92d837ee129e987ba875e7 by Akseli Lahtinen.
Committed on 04/04/2025 at 20:00.
Pushed by akselmo into branch 'master'.

KUrlNavigatorButton: Use arrow as separators

Due to the feedback of regular separator not being good enough for
separating folders, use an arrow separator instead.

This also refactors bunch of code around the button, so it should
be easier to tweak over time.

Remove icons due to the added clutter. They would work with full height chevrons but those are not technically feasible at this time.
Related: bug 501587, bug 501575, bug 501582, bug 501706, bug 501803
FIXED-IN: 6.14

M  +38   -20   src/filewidgets/kurlnavigator.cpp
M  +15   -0    src/filewidgets/kurlnavigator.h
M  +75   -70   src/filewidgets/kurlnavigatorbutton.cpp
M  +5    -3    src/filewidgets/kurlnavigatorbutton_p.h
M  +10   -14   src/filewidgets/kurlnavigatorbuttonbase.cpp
M  +1    -1    src/filewidgets/kurlnavigatortogglebutton.cpp

https://invent.kde.org/frameworks/kio/-/commit/7067680d23229b80dd92d837ee129e987ba875e7