Closed jyun261 closed 2 years ago
Changes look good! The only thing I see that might be wrong is definition queued. I thought when a user double clicks a word to open the definition screen, that should set definition queued = true. But with our code, definition queued is only true when the user clicks the definition on the definition screen. Or is this how it is supposed to be?
Changes look good! The only thing I see that might be wrong is definition queued. I thought when a user double clicks a word to open the definition screen, that should set definition queued = true. But with our code, definition queued is only true when the user clicks the definition on the definition screen. Or is this how it is supposed to be?
Ah sorry, didn't notice that. It's definitely not supposed to be like that. Thanks for catching my mistake.
move current quiz data from sessionStorage to the database (this means every word in wordBank has a new field value: starNumber that keeps track of current quiz progress)
fix single click being registered for a double click (word audio doesn't get played twice while trying to cue up a definition)
fix bug where quiz answers can be selected multiple times
fix bug where all words are highlightable on screen load
change html() to append() as much as I can because this method presents a security risk (or so I've been told)
fix bug where quiz choices can be selected after the player has already made a choice
made it so the quiz is repeated if the player gets it wrong
fix bug where semicolon would still get added to the last derivative word even if it's the last
some very small optimizations
As a side note, the database has been cleaned so that only words with the updated field values are present.