Option to show reminder instead of revealing answer

[dae]

Problem
when studying cards with auto advance feature enabled, it suddenly flip the card after the selected time in the settings, this (sudden flip) is actually annoying, because sometime I’m still reading the card and it flips suddenly.

Originally reported on https://forums.ankiweb.net/t/request-auto-alert-feature-in-auto-advance-mode/42198

I'm imagining a drop-down option like: "After showing question: [reveal answer|show reminder]"

[escribapetrus]

I'm planning to do this next. If someone's working on it or has a solution planned, please leave a comment. 🐐 🍄

UUPDATE: I haven't found the time yet to implement this. It's on my radar, but if anyone wants to do it, be my guest.

[Loudwig]

Hello, @escribapetrus are you still working on it ? I would like to give it a try to. Is it ok with you @dae also ? Thanks

[dae]

No objections from me. When implementing it, please use an enum to store the drop-down option, so we can add more options in the future if we need to.

[escribapetrus]

@Loudwig please go for it. I'm feeling sick today, and have no energy to do this.

[Loudwig]

Anki_Time_Extend_Auto_Update_video480p.mov.zip
Here is what I implemented, tell me what you think about it. Does it correspond to what you expected ? @dae @escribapetrus

[dae]

Unfortunately it's a bit different. Currently, if the user sets a timer on the question side, it shows the answer. The desired feature here is a dropdown choice, so that instead of revealing the answer, Anki would show a pop-up (tooltip) message, like it does when the answer action is "show reminder". There is no need for a button to extend the timer.

[Loudwig]

Ah ! Ok i understand the feature now. I'll do it. It will be faster since i already looked at this part of the code. @dae
But isn't it weird not to have an autoupdate of the question if we are in a autoupdate mode ?
I keep the version with the button to extend the timer if you want to implement it. I think it could also be a solution to deal with original issue.

[Loudwig]

Here is the new feature !! Is it what you were thinking about ? @dae

[dae]

Thank you, that's close. Could you submit a PR to this repo so I can add some comments to the code? There are a few minor things that need tweaking, like using "Title Case" for the options to match the capitalization of the other options, and a few tweaks to the wording of the docs.

[Loudwig]

Yes sure ! I did a PR but it failed merging with the main i believe.. ! Do i need to merge it with main on my own repository before doing a PR ? @dae