Bug 468755

Summary:	[Start Page] The Places sidebar doesn't restore the changed icon size
Product:	[Applications] gwenview	Reporter:	popov895 <popov895>
Component:	general	Assignee:	Gwenview Bugs <gwenview-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugseforuns, nate, popov895
Priority:	NOR
Version First Reported In:	Git (add output of "git log -1 --oneline" to description)
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/graphics/gwenview/commit/41ce55c6c71c150ba3eb3c6a41704c1905bf2177	Version Fixed/Implemented In:	23.04.1
Sentry Crash Report:

Description popov895 2023-04-21 10:33:18 UTC

SUMMARY

When the Gwenview starts, the Places sidebar of the start page always uses the same icon size (Large - 32px), no matter what size I set before.

STEPS TO REPRODUCE

1. Open Gwenview or go to Start Page (Alt+Home)
2. Change icon size of the Places sidebar to Small (right click > Icon Size > Small (16x16))
3. Restart application

OBSERVED RESULT

The size of the icons in the Places sidebar is Large (32x32)

EXPECTED RESULT

The size of the icons in the Places sidebar is Small (16x16)

SOFTWARE/OS VERSIONS

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.27.80
KDE Frameworks Version: 5.104.0
Qt Version: 5.15.8
Gwenview Version: be7269c2 (HEAD -> master, upstream/master, origin/master, origin/HEAD) GIT_SILENT Sync po/docbooks with svn

Comment 1 Bug Janitor Service 2023-04-21 10:41:01 UTC

A possibly relevant merge request was started @ https://invent.kde.org/graphics/gwenview/-/merge_requests/197

Comment 2 Patrick Silva 2023-04-21 12:02:46 UTC

Possibly duplicate of bug 463141

Comment 3 popov895 2023-04-21 12:46:31 UTC

(In reply to Patrick Silva from comment #2)
> Possibly duplicate of bug 463141

Definitely yes. Sorry I didn't search well.

Comment 4 popov895 2023-04-21 14:19:24 UTC

Git commit 8de9e4cba77c41bd25b3b8140ade342f5983911c by Eugene Popov.
Committed on 21/04/2023 at 10:40.
Pushed by ngraham into branch 'master'.

Don't set the default icon size for KFilePlacesView in *.ui file

This causes the KFilePlacesView to always use this icon size (a bug in the implementation of the KFilePlacesView).

M  +0    -6    app/startmainpage.ui

https://invent.kde.org/graphics/gwenview/commit/8de9e4cba77c41bd25b3b8140ade342f5983911c

Comment 5 Nate Graham 2023-04-21 15:42:26 UTC

Git commit 41ce55c6c71c150ba3eb3c6a41704c1905bf2177 by Nate Graham, on behalf of Eugene Popov.
Committed on 21/04/2023 at 15:42.
Pushed by ngraham into branch 'release/23.04'.

Don't set the default icon size for KFilePlacesView in *.ui file

This causes the KFilePlacesView to always use this icon size (a bug in the implementation of the KFilePlacesView).


(cherry picked from commit 8de9e4cba77c41bd25b3b8140ade342f5983911c)

M  +0    -6    app/startmainpage.ui

https://invent.kde.org/graphics/gwenview/commit/41ce55c6c71c150ba3eb3c6a41704c1905bf2177