mirror of
https://github.com/kovidgoyal/calibre.git
synced 2025-07-08 18:54:09 -04:00
Fix deleting a book from both the library and the device causing the next selected book to be incorrect, when the library is sorted by the On Device column. Fixes #1404629 [Deleting a book both in device and library makes cursor jump](https://bugs.launchpad.net/calibre/+bug/1404629)
This commit is contained in:
parent
febd725abe
commit
c33d682de7
@ -332,6 +332,7 @@ class DeleteAction(InterfaceAction):
|
|||||||
|
|
||||||
def do_library_delete(self, to_delete_ids):
|
def do_library_delete(self, to_delete_ids):
|
||||||
view = self.gui.current_view()
|
view = self.gui.current_view()
|
||||||
|
next_id = view.next_id
|
||||||
# Ask the user if they want to delete the book from the library or device if it is in both.
|
# Ask the user if they want to delete the book from the library or device if it is in both.
|
||||||
if self.gui.device_manager.is_device_present:
|
if self.gui.device_manager.is_device_present:
|
||||||
on_device = False
|
on_device = False
|
||||||
@ -360,7 +361,6 @@ class DeleteAction(InterfaceAction):
|
|||||||
'removed from your calibre library. Are you sure?')%len(to_delete_ids)
|
'removed from your calibre library. Are you sure?')%len(to_delete_ids)
|
||||||
+'</p>', 'library_delete_books', self.gui):
|
+'</p>', 'library_delete_books', self.gui):
|
||||||
return
|
return
|
||||||
next_id = view.next_id
|
|
||||||
if len(to_delete_ids) < 5:
|
if len(to_delete_ids) < 5:
|
||||||
try:
|
try:
|
||||||
view.model().delete_books_by_id(to_delete_ids)
|
view.model().delete_books_by_id(to_delete_ids)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user