ankidroid / Anki-Android

AnkiDroid: Anki flashcards on Android. Your secret trick to achieve superhuman information retention.
GNU General Public License v3.0
8.71k stars 2.24k forks source link

Mathjax and cloze deletion bug duplicate portions of the card #5703

Closed mangelozzi closed 4 years ago

mangelozzi commented 4 years ago
Reproduction Steps
  1. Create a cloze card with: \(A\) {{c1::x}} \(B\) {{c1::y}} \(C\) {{c1::z}} \(D\)
Expected Result

A x B y C z D

Actual Result

A x B y C z D y C z D

Works fine on the desktop version, but quite a few of my cards in the android app have duplicate chunks of the card, and have simplified into this easily reproducible test case. As you can see in the above case "y C z D" appears twice.

Might be related to #5552 (closed)

Research

Enter an [ x ] character to confirm the points below:

[x] I have read the support page and am reporting a bug or enhancement request specific to AnkiDroid

[x] I have checked the manual and the FAQ and could not find a solution to my issue

[x] I have searched for similar existing issues here and on the user forum

mikehardy commented 4 years ago

Have you tried the latest alphas?https://docs.ankidroid.org/manual.html#betaTesting

mangelozzi commented 4 years ago

I haven't tried alpha testing, but I did first make sure I was on the latest offered by the app store.

mikehardy commented 4 years ago

That is unfortunately not specific enough, as the app store (really the "play store", apple is the app store - but I do know what you mean) can offer the stable, beta or alpha version. If you sign up for betas in the play store app and do nothing else, you'll start getting the betas.

This one should be fixed in 2.9beta2 according to the linked issue though, so if you are definitely on that version or higher (you can see at the bottom of advanced preferences) then the bug still exists.

If it does exist, then any future fix will be in an alpha - to get those you'd need to also follow the link above and join a special google group signalling alpha interest. There is zero traffic on that list, but it is important because the play store uses membership in that list to offer you the alphas or not. Then you can test it

But first we need to know your exact current version to make sure it's reproduced well

I'll also tag in @adamwolf who has so far generously been our mathjax expert :-)

adamwolf commented 4 years ago

If this is still a bug, I can jump in and make sure it's fixed. I am currently drowning in work that I delayed over the holidays, so before I do that, could you please check that you are on 2.9beta2 or newer? If not, please update to that or newer.

Thank you so much for including a good example, if this is something I have to jump in and fix that will make this relatively quick.

On Tue, Jan 7, 2020 at 1:55 PM Mike Hardy notifications@github.com wrote:

That is unfortunately not specific enough, as the app store (really the "play store", apple is the app store - but I do know what you mean) can offer the stable, beta or alpha version. If you sign up for betas in the play store app and do nothing else, you'll start getting the betas.

This one should be fixed in 2.9beta2 according to the linked issue though, so if you are definitely on that version or higher (you can see at the bottom of advanced preferences) then the bug still exists.

If it does exist, then any future fix will be in an alpha - to get those you'd need to also follow the link above and join a special google group signalling alpha interest. There is zero traffic on that list, but it is important because the play store uses membership in that list to offer you the alphas or not. Then you can test it

But first we need to know your exact current version to make sure it's reproduced well

I'll also tag in @adamwolf https://github.com/adamwolf who has so far generously been our mathjax expert :-)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ankidroid/Anki-Android/issues/5703?email_source=notifications&email_token=AAAIWYILJQ5MZD2PH7I7VQTQ4TM3HA5CNFSM4KD2YAO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIKCISY#issuecomment-571745355, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAIWYJ5X47KW2IMLAAOOX3Q4TM3HANCNFSM4KD2YAOQ .

mangelozzi commented 4 years ago

Comment deleted for now... just waiting a few hours to see if joining the alpha group and the update becomes available. BTW to reproduce the bug, it's really quick. One just needs to create a cloze card and paste in:

`\(A\) {{c1::x}} \(B\) {{c1::y}} \(C\) {{c1::z}} \(D\)`
mikehardy commented 4 years ago

Sometimes if you go to the "beta" tab at the top of the play store app, you can see the updates earlier than if you wait for it's daily-or-about-daily update cycle to run

mangelozzi commented 4 years ago

It is fixed in 2.10alpha30. Thanks guys.

mikehardy commented 4 years ago

Yes! Thanks for coming back and updating