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

Méven <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/syst |https://invent.kde.org/syst
                   |em/dolphin/-/commit/a2a1cb6 |em/dolphin/-/commit/57c6b10
                   |1703da371eb8b4995bc78075d95 |e1a225c308dc25599ec49abca1a
                   |180991                      |e6d958

--- Comment #4 from Méven <[email protected]> ---
Git commit 57c6b10e1a225c308dc25599ec49abca1ae6d958 by Méven Car.
Committed on 23/08/2026 at 13:31.
Pushed by meven into branch 'release/26.08'.

dolphinitemlistview: do not store the fallback icon size in the cache

updateGridSe() bound a reference to m_iconSize or m_previewSize and wrote the
current mode's configured size into it when that was still unset. The cache
holds
the per-folder zoom, one value for a folder that every view mode shares, so a
size
that was only a fallback reached the other modes as though the folder carried
it.

A view starts out in the details layout, so the cache was seeded from
DetailsModeSettings. Opening a folder in icons view with per-folder view
properties and then turning previews off rendered it at 16 rather than the 32
that
icons view is configured with, because turning previews off selects the other
cached size and no mode change recomputes it.

The fallback itself comes from commit 3ce18dd17 ("views: Never let the item
layout
use a zero icon size"), which meant to cover the preview toggle too. Storing it
is
what left that case short.

Read the fallback without storing it, so the cache stays unset until
setZoomLevel() puts a chosen zoom there.
FIXED-IN: 26.08.1
(cherry picked from commit a2a1cb61703da371eb8b4995bc78075d95180991)

A  +329  -0    src/tests/dolphinviewtest.cpp     [License: GPL(v2.0+)]
M  +5    -8    src/views/dolphinitemlistview.cpp

https://invent.kde.org/system/dolphin/-/commit/57c6b10e1a225c308dc25599ec49abca1ae6d958

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

Reply via email to