Fix: Limit missing media snackbar to 2 times in new study screen #20357
Fix: Limit missing media snackbar to 2 times in new study screen #20357Mahakbajpai wants to merge 1 commit intoankidroid:mainfrom
Conversation
…atch old study behaviour
|
First PR! 🚀 We sincerely appreciate that you have taken the time to propose a change to AnkiDroid! Please have patience with us as we are all volunteers - we will get to this as soon as possible. |
|
Hi, I'm not reviewing but since I've been waited to be assigned to this issue, so I'm commenting on it. It looks like the logic has changes, so the unit tests need to be updated accordingly! I also recommend keeping the copyright. When I worked on this, I just added Additionally it'd be better if the commit length could be a bit shorter 😉 |
Purpose / Description
In the new study screen, the missing media snackbar was being displayed repeatedly for every missing media event. This behavior was inconsistent with the old study screen.
This PR aligns the new study screen behavior with the old study screen by limiting the missing media snackbar display to a maximum of 2 times per card.
Fixes
Fixes #20347
Approach
hasExecutedstate fromMediaErrorHandlermissingMediaCountMAX_DISPLAY_TIMES = 2How Has This Been Tested?
Tested manually on:
Result: