langfingaz/anki
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1154 from RumovZ/fix-skip-dec

Browse Source 
Fix skip_if_selection_is_empty() for triggers
This commit is contained in:
Damien Elmes 2021-04-26 09:21:42 +10:00 committed by GitHub
commit f77983fc9f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
qt/aqt/browser/browser.py
View File

@ -72,6 +72,7 @@ from aqt.utils import (
showInfo,
showWarning,
skip_if_selection_is_empty,
skip_if_selection_is_empty_on_trigger,
tooltip,
tr,
)
@ -160,11 +161,8 @@ class Browser(QMainWindow):
f.actionCreateFilteredDeck.setShortcuts(["Ctrl+G", "Ctrl+Alt+G"])
# notes
qconnect(f.actionAdd.triggered, self.mw.onAddCard)
qconnect(f.actionAdd_Tags.triggered, lambda: self.add_tags_to_selected_notes())
qconnect(
f.actionRemove_Tags.triggered,
lambda: self.remove_tags_from_selected_notes(),
)
qconnect(f.actionAdd_Tags.triggered, self.add_tags_to_selected_notes)
qconnect(f.actionRemove_Tags.triggered, self.remove_tags_from_selected_notes)
qconnect(f.actionClear_Unused_Tags.triggered, self.clear_unused_tags)
qconnect(f.actionToggle_Mark.triggered, self.toggle_mark_of_selected_notes)
qconnect(f.actionChangeModel.triggered, self.onChangeModel)
@ -189,7 +187,7 @@ class Browser(QMainWindow):
qconnect(
f.actionBlue_Flag.triggered, lambda: self.set_flag_of_selected_cards(4)
)
qconnect(f.actionExport.triggered, lambda: self._on_export_notes())
qconnect(f.actionExport.triggered, self._on_export_notes)
# jumps
qconnect(f.actionPreviousCard.triggered, self.onPreviousCard)
qconnect(f.actionNextCard.triggered, self.onNextCard)
@ -574,7 +572,7 @@ where id in %s"""
# Misc menu options
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def onChangeModel(self) -> None:
ChangeModel(self, self.oneModelNotes())
@ -619,7 +617,7 @@ where id in %s"""
# Card deletion
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
def delete_selected_notes(self) -> None:
# ensure deletion is not accidentally triggered when the user is focused
# in the editing screen or search bar
@ -641,7 +639,7 @@ where id in %s"""
# Deck change
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def set_deck_of_selected_cards(self) -> None:
from aqt.studydeck import StudyDeck
@ -670,7 +668,7 @@ where id in %s"""
# Tags
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def add_tags_to_selected_notes(
self,
@ -683,7 +681,7 @@ where id in %s"""
parent=self, note_ids=self.selected_notes(), space_separated_tags=tags
).run_in_background(initiator=self)
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def remove_tags_from_selected_notes(self, tags: Optional[str] = None) -> None:
"Shows prompt if tags not provided."
@ -730,7 +728,7 @@ where id in %s"""
# Exporting
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
def _on_export_notes(self) -> None:
cids = self.selectedNotesAsCards()
ExportDialog(self.mw, cids=list(cids))
@ -781,7 +779,7 @@ where id in %s"""
# Scheduling
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def reposition(self) -> None:
if self.card and self.card.queue != QUEUE_TYPE_NEW:
@ -793,7 +791,7 @@ where id in %s"""
):
op.run_in_background()
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def set_due_date(self) -> None:
if op := set_due_date_dialog(
@ -803,7 +801,7 @@ where id in %s"""
):
op.run_in_background()
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def forget_cards(self) -> None:
forget_cards(
@ -814,7 +812,7 @@ where id in %s"""
# Edit: selection
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def selectNotes(self) -> None:
nids = self.selected_notes()
@ -863,7 +861,7 @@ where id in %s"""
# Edit: replacing
######################################################################
@skip_if_selection_is_empty
@skip_if_selection_is_empty_on_trigger
@ensure_editor_saved_on_trigger
def onFindReplace(self) -> None:
FindAndReplaceDialog(self, mw=self.mw, note_ids=self.selected_notes())

6
qt/aqt/utils.py
View File

@ -1022,6 +1022,12 @@ def skip_if_selection_is_empty(func: Callable) -> Callable:
return decorated
def skip_if_selection_is_empty_on_trigger(func: Callable) -> Callable:
"""Like skip_if_selection_is_empty(), but tells Qt this function takes no args."""
return pyqtSlot()(skip_if_selection_is_empty(func)) # type: ignore
class KeyboardModifiersPressed:
"Util for type-safe checks of currently-pressed modifier keys."