ucfopen / crossword-materia-widget Goto Github PK

When using the crossword widget, I keep receiving the error ReferenceError: combinedClueText is not defined. To see this error in the console, go to https://materia.ucf.edu/widgets/10-crossword and accept the first two hints.

• Tab index is overridden with discrete values > 1, which should be reverted to a default tab flow
• Many DOM elements are not representative of their actual role, or do not have appropriate roles applied
• Focusable element navigation and flow is inconsistent with ARIA standards
• Not all focusable elements have a :focus style applied
• Improvements should be made to clue selection and board navigation

Currently it's possible for just about any unicode character to be used in Crossword. A crossword puzzle can be created in Russian, for example - the characters can be provided to the creator and a puzzle will be generated properly, and (it looks like) scoring will work properly as well.

Currently the Crossword player has an explicit list of characters that are considered 'allowed input'. These include the 0-9 digits, all English letters, and a number of accented characters common to western European languages. Any character included in a puzzle that is not in that list will be pre-filled and non-editable.

Consider keeping the current set of 'allowed characters' as a 'standard' set but also scraping additional characters from the qset to generate a 'all possible characters for this puzzle' list. Or potentially allowing widget creators to configure a list of allowed/disallowed characters.

While the player does a good job of preventing whitespace in most student inputs, there is one circumstance where it isn't explicitly prevented: when a user is focused on a character slot that was previously filled out, for example where words intersect. A user can enter whitespace characters and then use the arrow keys or cursor to manually select the next character slot, which results in malformed response strings being sent to Materia for scoring.

To reproduce:

1. Load up a crossword instance
2. Attempt to select different words by selecting the hint on the right-hand side (not the button, but the entire hint)

Depending on the index of the word in the list, some hints may not be selectable, or an entirely different hint may be selected instead. The selection will bounce around unpredictably.

The React conversion requires that we remove the uses of print.js and classList.min.js in the score screen of the widget at src > scoreScreen.html.

• Fix for scoring dashes
• Fix for different "special characters" appearing than selected
• Allow enter key to be pressed to jump to next word
• Fix for words with special characters as first letter not being able to be selected
• _checkIfDone() fix

Working on a fix for words that are connected by their first letters. This intersection causes two numberlabels to be applied in the same place, making them overlap. Simple solution is to make vertical numberlabels left-aligned. Better solution is to change how the numbers are created so that they share the same number.

There have been some UI changes to the widget, but the screenshots haven't been updated to match.

Sample widget: https://materia.ucf.edu/preview/493/spinal-cord-disorders