Bug 409339

Summary: "Delete thumbnail cache folder on exit" description text is incorrect
Product: [Applications] gwenview Reporter: Zachary Mazza <zwjmazza>
Component: generalAssignee: Gwenview Bugs <gwenview-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: nate, yurchor
Priority: NOR    
Version: 19.04.2   
Target Milestone: ---   
Platform: Kubuntu   
OS: Linux   
Latest Commit: Version Fixed In: 19.08.0
Attachments: Screenshot of the incorrect description.

Description Zachary Mazza 2019-06-30 19:20:28 UTC
Created attachment 121243 [details]
Screenshot of the incorrect description.

SUMMARY
The text description under the option "Cache: Delete thumbnail cache folder on exit" warns about a nonexistent location of the thumbnail cache (~/.thumbnails), rather than the current one (~/.cache/thumbnails or $XDG_CACHE_HOME/thumbnails).

STEPS TO REPRODUCE
1. Configure Gwenview
2. Navigate to the advanced section

OBSERVED RESULT
The text under the option warns that the folder ~/.thumbnails will be deleted, despite no longer being the storage location for thumbnails.

EXPECTED RESULT
The text should instead show either "~/.cache/thumbnails" or $XDG_CACHE_HOME/thumbnails". 
Alternatively, it could say "this will delete the folder named thumbnails in your cache folder".

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Kubuntu 19.04
(available in About System)
KDE Plasma Version: 5.16.2
KDE Frameworks Version: 5.59.0
Qt Version: 5.12.2
Comment 1 Nate Graham 2019-06-30 23:00:26 UTC
Git commit 1b6ad2e9fb288fb8b70e1ab2e4b305a0fd7bd1be by Nate Graham.
Committed on 30/06/2019 at 23:00.
Pushed by ngraham into branch 'master'.

Adjust Low Resource Mode's explanatory text to be accurate
FIXED-IN: 19.08.0

M  +1    -1    app/advancedconfigpage.ui
M  +1    -1    doc/index.docbook

https://commits.kde.org/gwenview/1b6ad2e9fb288fb8b70e1ab2e4b305a0fd7bd1be
Comment 2 Zachary Mazza 2019-07-01 20:29:43 UTC
I also think that the period in ".thumbnails" should be removed as it's no longer a dot folder.

Thanks for fixing this.
Comment 3 Nate Graham 2019-07-10 18:38:29 UTC
Yuri fixed this in https://cgit.kde.org/gwenview.git/commit/?id=6c06c91713ffe58c0a2b93954d0b833b852dfe4c.

Thanks Yuri!