Bug 270435 - Dolphin should not clear selection when deactivating preview
Summary: Dolphin should not clear selection when deactivating preview
Status: RESOLVED FIXED
Alias: None
Product: dolphin
Classification: Applications
Component: general (show other bugs)
Version: 16.12.2
Platform: Ubuntu Linux
: NOR minor
Target Milestone: ---
Assignee: Peter Penz
URL:
Keywords: reproducible
Depends on:
Blocks:
 
Reported: 2011-04-08 14:01 UTC by Alvaro Manuel Recio Perez
Modified: 2011-04-08 20:48 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.7.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alvaro Manuel Recio Perez 2011-04-08 14:01:43 UTC
Version:           unspecified (using KDE 4.6.2) 
OS:                Linux

When some files and or folders are selected and the user clicks the "Preview" button, the selection is kept after the view is updated. However, if the user clicks "Preview" again, to disable it, the selection is cleared.

Dolphin should be consistent in this behaviour and, in my opinion, not clearing the selection is the most appropriate option.

Reproducible: Always

Steps to Reproduce:
1. Open Dolphin and select some files.
2. Click Preview to activate it.
3. Notices that the selection is kept.
4. Click Preview to disable it.

Actual Results:  
The selection is cleared.

Expected Results:  
The selection should not be cleared.

OS: Linux (x86_64) release 2.6.35-28-generic
Compiler: cc
Comment 1 Frank Reininghaus 2011-04-08 20:04:55 UTC
Thanks for the bug report! I can confirm the issue in master.
Comment 2 Peter Penz 2011-04-08 20:24:50 UTC
The issue has the same root cause like bug 270437 - will try to fix it during this week.
Comment 3 Peter Penz 2011-04-08 20:48:45 UTC
Git commit 41854146040f8638fffb13b20a75f6ecda2837a6 by Peter Penz.
Committed on 08/04/2011 at 20:52.
Pushed by ppenz into branch 'master'.

KFilePreviewGenerator: Prevent unnecessary reloading of directory

When turning off previews the reloading of the directory is
unnecessary and has side effects:
- The selection gets cleared
- The position of the view might get reset

BUG: 270437
BUG: 270435
FIXED-IN: 4.7.0

M  +17   -13   kfile/kfilepreviewgenerator.cpp     

http://commits.kde.org/kdelibs/41854146040f8638fffb13b20a75f6ecda2837a6