Bug 409339 - "Delete thumbnail cache folder on exit" description text is incorrect
Summary: "Delete thumbnail cache folder on exit" description text is incorrect
Status: RESOLVED FIXED
Alias: None
Product: gwenview
Classification: Applications
Component: general (show other bugs)
Version: 19.04.2
Platform: Kubuntu Linux
: NOR minor
Target Milestone: ---
Assignee: Gwenview Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-30 19:20 UTC by Zachary Mazza
Modified: 2019-07-10 18:38 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 19.08.0


Attachments
Screenshot of the incorrect description. (18.69 KB, image/png)
2019-06-30 19:20 UTC, Zachary Mazza
Details

Note You need to log in before you can comment on or make changes to this bug.
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!