| Summary: | Keyboard shortcut Ctrl+0 should reset to user set default icon size and not the smallest icon size | ||
|---|---|---|---|
| Product: | [Applications] dolphin | Reporter: | SigHunter <sighunter> |
| Component: | view-engine: details mode | Assignee: | Dolphin Bug Assignee <dolphin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | dolphin-bugs-null, fanzhuyifan, filip.kendes1, madness742 |
| Priority: | NOR | Keywords: | qt6 |
| Version First Reported In: | 24.08.2 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/system/dolphin/-/commit/70cd6c09296033f7de847d7006062e802b6e6473 | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
|
Description
SigHunter
2024-03-03 16:10:45 UTC
to keep this updated: this is still the case with latest Dolphin 24.08.2 in short: "reset zoom level" keyboard shortcut should set icon size to user configured default, not smallest icon size Tested on Operating System: Gentoo Linux 2.15 KDE Plasma Version: 6.2.1 KDE Frameworks Version: 6.7.0 Qt Version: 6.7.3 Kernel Version: 6.11.3-gentoo-dist (64-bit) Graphics Platform: Wayland Processors: 20 × Intel® Core™ i9-10850K CPU @ 3.60GHz Memory: 31.2 GiB of RAM Graphics Processor: NVIDIA GeForce RTX 3080 Ti/PCIe/SSE2 Can confirm in Dolphin 24.12.0 It affects all view modes A possibly relevant merge request was started @ https://invent.kde.org/system/dolphin/-/merge_requests/1080 Git commit 476cb0af843a26ebdd509c5b400532d20f71dcb2 by Méven Car, on behalf of weinan li. Committed on 29/10/2025 at 10:31. Pushed by meven into branch 'master'. dolphinView: resetZoomLevel: reset to user defined default zoom Instead of ViewModeSettings default mode. The original resetZoomLevel() implementation used settings.useDefaults(true) to force reading the system's hardcoded default icon size (typically 16px), ignoring the user's custom default size configured in Dolphin's View settings. This caused Ctrl+0 to reset icons to the minimum size instead of the user's preferred default. This patch removes the useDefaults(true) call, allowing ViewModeSettings::iconSize() to directly retrieve the user-configured default icon size. Ctrl+0 now correctly resets to the size the user set in Dolphin's settings, aligning with expected behavior. M +2 -4 src/views/dolphinview.cpp https://invent.kde.org/system/dolphin/-/commit/476cb0af843a26ebdd509c5b400532d20f71dcb2 Git commit 70cd6c09296033f7de847d7006062e802b6e6473 by Méven Car. Committed on 02/11/2025 at 19:05. Pushed by meven into branch 'release/25.08'. dolphinView: resetZoomLevel: reset to user defined default zoom Instead of ViewModeSettings default mode. The original resetZoomLevel() implementation used settings.useDefaults(true) to force reading the system's hardcoded default icon size (typically 16px), ignoring the user's custom default size configured in Dolphin's View settings. This caused Ctrl+0 to reset icons to the minimum size instead of the user's preferred default. This patch removes the useDefaults(true) call, allowing ViewModeSettings::iconSize() to directly retrieve the user-configured default icon size. Ctrl+0 now correctly resets to the size the user set in Dolphin's settings, aligning with expected behavior. (cherry picked from commit 476cb0af843a26ebdd509c5b400532d20f71dcb2) Co-authored-by: weinan li <liweinan@kylinos.cn> M +2 -4 src/views/dolphinview.cpp https://invent.kde.org/system/dolphin/-/commit/70cd6c09296033f7de847d7006062e802b6e6473 |