Bug 307522 - The hidden option "EnlargeSmallPreviews" does not work any more
Summary: The hidden option "EnlargeSmallPreviews" does not work any more
Status: RESOLVED FIXED
Alias: None
Product: dolphin
Classification: Applications
Component: view-engine: general (show other bugs)
Version: 2.1
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Frank Reininghaus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-28 10:42 UTC by Frank Reininghaus
Modified: 2012-10-18 10:57 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
First draft of a patch (3.30 KB, patch)
2012-09-29 19:00 UTC, Frank Reininghaus
Details
Updated patch (2.17 KB, patch)
2012-10-03 08:24 UTC, Frank Reininghaus
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Frank Reininghaus 2012-09-28 10:42:41 UTC
In

https://projects.kde.org/projects/kde/kde-baseapps/repository/revisions/99e4eb0f3a58c53277b099114a5fd8945937fb9a

a hidden option "EnlargeSmallPreviews" has been added to the [General] section of dolphinrc. When this option is set to false, images which are smaller than the preview size should not be scaled up (this feature had been requested by Nuno). However, this does not work currently, neither in the 4.9 branch, nor in master.


Reproducible: Always
Comment 1 Frank Reininghaus 2012-09-29 19:00:06 UTC
Created attachment 74235 [details]
First draft of a patch

The commit that broke it appears to be 2069c200211c93e401c3d19850f41eade09d64be

This is what I've achieved so far: Icons are not scaled up any more when the settings is disabled, the icons are centered at the bottom, and no frame is drawn. However, strange arrows are painted next to some icons for me. I have no idea why :-(
Comment 2 Frank Reininghaus 2012-10-03 08:18:03 UTC
Git commit 0b72170cd525e35db11dd48b230da9e567bb99b4 by Frank Reininghaus.
Committed on 03/10/2012 at 10:14.
Pushed by freininghaus into branch 'KDE/4.9'.

Fix the hidden "EnlargeSmallPreviews" option

This option, introduced in 99e4eb0f3a58c53277b099114a5fd8945937fb9a, did
not work any more since 2069c200211c93e401c3d19850f41eade09d64be.

M  +4    -1    dolphin/src/views/dolphinview.cpp

http://commits.kde.org/kde-baseapps/0b72170cd525e35db11dd48b230da9e567bb99b4
Comment 3 Frank Reininghaus 2012-10-03 08:24:30 UTC
Created attachment 74313 [details]
Updated patch

This is the patch which makes the icons centered at the bottom without frame, rebased on my recent commit. Unfortunately, the issue with the strange arrows remains.
Comment 4 Frank Reininghaus 2012-10-18 10:57:26 UTC
It seems that the strange arrows appear in every case, even with the current state of the 4.9 branch...

Nuno, the option should be working now in the 4.9 branch, therefore, I'm closing this report. Some things that I can do additionally if you want:

a) Remove the frame if the 'EnlargeSmallPreviews' option is disabled
b) Do not center icons, but show them centered at the bottom if the option is disabled.

If you want one or both of these options, just add a comment here and I'll take care of it.