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

ensure multiple answer buttons don't get accepted when mashed in v3

Browse Source 
We want to preserve the backend checks, because they may also detect
bugs. So we do the debouncing on the frontend instead, transitioning
into a temporary state until the background op completes.

https://forums.ankiweb.net/t/anki-2-1-50-beta/15608/74
This commit is contained in:
Damien Elmes 2021-12-14 15:48:02 +10:00
parent 46b85d508a
commit 0d51b4db1f
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
qt/aqt/reviewer.py
View File

@ -121,7 +121,7 @@ class Reviewer:
self._answeredIds: list[CardId] = [] self._answeredIds: list[CardId] = []
self._recordedAudio: str | None = None self._recordedAudio: str | None = None
self.typeCorrect: str = None # web init happens before this is set self.typeCorrect: str = None # web init happens before this is set
self.state: str | None = None self.state: Literal["question", "answer", "transition", None] = None
self._refresh_needed: RefreshNeeded | None = None self._refresh_needed: RefreshNeeded | None = None
self._v3: V3CardInfo | None = None self._v3: V3CardInfo | None = None
self._state_mutation_key = str(random.randint(0, 2 ** 64 - 1)) self._state_mutation_key = str(random.randint(0, 2 ** 64 - 1))
@ -420,6 +420,7 @@ class Reviewer:
if sched.state_is_leech(answer.new_state): if sched.state_is_leech(answer.new_state):
self.onLeech() self.onLeech()
self.state = "transition"
answer_card(parent=self.mw, answer=answer).success( answer_card(parent=self.mw, answer=answer).success(
after_answer after_answer
).run_in_background(initiator=self) ).run_in_background(initiator=self)