[krusader] [Bug 389869] After deletion of selected files current item moves one down

[Peter Mühlenpfordt]

https://bugs.kde.org/show_bug.cgi?id=389869

Peter Mühlenpfordt  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/kru
   ||sader/dc38f3cfc6fd447723196
   ||7c03ea7fe76581d4e4c
 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #3 from Peter Mühlenpfordt  ---
Git commit dc38f3cfc6fd4477231967c03ea7fe76581d4e4c by Peter Mühlenpfordt.
Committed on 14/02/2018 at 06:48.
Pushed by muhlenpfordt into branch 'master'.

Fix setting new current item after file deletion

Summary:
After deleting one or more selected files and current item is not one
of these, the current item is moved one down.
This patch modifies `KrView::firstUnmarkedBelowCurrent()` to not make
an additional step down in finding the next unselected item.

Test Plan:
1. Create a folder with 5 files "file1".."file5"
2. Select "file1" and "file2"
3. Set current item to "file3"
4. Press {key Del} and delete selected files
-> The current item should stay on "file3"

Reviewers: #krusader, martinkostolny

Reviewed By: #krusader, martinkostolny

Subscribers: martinkostolny, asensi

Differential Revision: https://phabricator.kde.org/D10385

M  +4-2krusader/Panel/PanelView/krview.cpp
M  +1-1krusader/Panel/PanelView/krview.h
M  +2-1krusader/Panel/listpanel.cpp

https://commits.kde.org/krusader/dc38f3cfc6fd4477231967c03ea7fe76581d4e4c

-- 
You are receiving this mail because:
You are watching all bug changes.

[krusader] [Bug 389869] After deletion of selected files current item moves one down

[Toni Asensi Esteve]

https://bugs.kde.org/show_bug.cgi?id=389869

--- Comment #2 from Toni Asensi Esteve  ---
Note: This is related to
https://phabricator.kde.org/D10385

-- 
You are receiving this mail because:
You are watching all bug changes.

[krusader] [Bug 389869] After deletion of selected files current item moves one down

[Toni Asensi Esteve]

https://bugs.kde.org/show_bug.cgi?id=389869

Toni Asensi Esteve  changed:

   What|Removed |Added

 Status|UNCONFIRMED |CONFIRMED
 CC||toni.ase...@kdemail.net
 Ever confirmed|0   |1

--- Comment #1 from Toni Asensi Esteve  ---
Thanks, Peter. 

This didn't happen using Krusader 2.4.0-beta3.

-- 
You are receiving this mail because:
You are watching all bug changes.