ngraham added a comment.

  Thanks. To account for this behavioral change, I think we need to adjust the 
UI strings in the settings window as well. Did my suggestion above make sense?
  
    Thumbnails: [ ] Low resource usage mode
                    Don't store thumbnails on disk, and prefer lower-quality
                    thumbnails that are faster to load, when available
    
                    Be careful: blablabla

INLINE COMMENTS

> advancedconfigpage.ui:51
>       <property name="text">
> -         <string>Enable this option if you do not have a lot of disk 
> space.&lt;br/&gt;&lt;br/&gt;&lt;em&gt;Be careful:&lt;/em&gt; this will delete 
> the folder named &lt;filename&gt;.thumbnails&lt;/filename&gt; in your home 
> folder, deleting all thumbnails previously generated by Gwenview and other 
> applications.</string>
> +         <string>Enable this option if you do not have a lot of disk space. 
> This will avoid to store thumbnails on disk and will prefer embedded 
> thumbnails of lower quality that are faster to load, if 
> available.&lt;br/&gt;&lt;br/&gt;&lt;em&gt;Be careful:&lt;/em&gt; this will 
> delete the folder named &lt;filename&gt;.thumbnails&lt;/filename&gt; in your 
> home folder, deleting all thumbnails previously generated by Gwenview and 
> other applications.</string>
>       </property>

`avoid to store` -> `avoid storing`

> index.docbook:849
>                  <listitem>
> -                    <para>Enable the <guilabel>Delete thumbnail cache folder 
> on exit</guilabel> option if you do not have a lot of disk space.</para>
> +                    <para>Enable the <guilabel>Delete thumbnail cache folder 
> on exit</guilabel> option if you do not have a lot of disk space. This will 
> avoid to store thumbnails on disk and will prefer embedded thumbnails of 
> lower quality that are faster to load, if available.</para>
>                      <para>Be careful: This will delete the folder named 
> <filename>.thumbnails</filename> in your home folder, deleting all thumbnails 
> previously generated by &gwenview; and other applications.</para>

Same ^^

REVISION DETAIL
  https://phabricator.kde.org/D21330

To: tommo, #gwenview
Cc: yurchor, kde-doc-english, ngraham, #gwenview, tommo, gennad, clintmoyer, 
tdarboux, huoni, muhlenpfordt, rkflx, skadinna

Reply via email to