Summary: | Long folder names create too much padding in breadcrumb | ||
---|---|---|---|
Product: | [Applications] dolphin | Reporter: | Squeaky Pancakes <squeakypancakes> |
Component: | bars: location | Assignee: | Dolphin Bug Assignee <dolphin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | bugseforuns, kfm-devel, mail |
Priority: | NOR | ||
Version: | 20.08.0 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/frameworks/kio/commit/c1e658c723fc2c5a4b7d9a03f193c07a909877af | Version Fixed In: | 5.74 |
Attachments: |
too much padding
normal amount of padding |
Created attachment 131028 [details]
normal amount of padding
Can confirm this behavior with Dolphin 20.11.70. Not sure why this padding is even dependant on the length of the folder name. A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kio/-/merge_requests/109 Git commit c1e658c723fc2c5a4b7d9a03f193c07a909877af by David Faure, on behalf of Ismael Asensio. Committed on 22/08/2020 at 11:48. Pushed by dfaure into branch 'master'. [filewidgets] Fix KUrlNavigatorButton padding on breadcrumb Use `QFontMetrics::size(Qt::SingleLine, text).width()` instead of `QFontMetrics::boundingRect(text).width()`. As suggested by @dfaure, from an empiric point of view, this seems to be a better replacement for `QFontMetrics::width()` when applied to strings than the other two alternatives. See also https://invent.kde.org/system/dolphin/-/merge_requests/22 FIXED-IN: 5.74 M +2 -2 src/filewidgets/kurlnavigatorbutton.cpp https://invent.kde.org/frameworks/kio/commit/c1e658c723fc2c5a4b7d9a03f193c07a909877af |
Created attachment 131027 [details] too much padding SUMMARY The padding in the breadcrumb menu around folders with long names is way to big. STEPS TO REPRODUCE 1. Create a folder with a long name 2. Go into folder 3. profit OBSERVED RESULT EXPECTED RESULT Operating System: Artix Linux KDE Plasma Version: 5.19.4 KDE Frameworks Version: 5.73.0 Qt Version: 5.15.0 Kernel Version: 5.4.54-1-lts OS Type: 64-bit Processors: 4 × Intel® Core™ i5-3210M CPU @ 2.50GHz Memory: 15.4 GiB of RAM Graphics Processor: Mesa DRI Intel® HD Graphics 4000 ADDITIONAL INFORMATION