Skip to content

Conversation

@hannahbergam
Copy link
Contributor

@hannahbergam hannahbergam commented Jan 28, 2026

This wraps the fontAwesomeIcons in buttons, which makes them keyboard navigable. It also adds aria labels to each so a students knows they are toggling to see fish or trash.

I tested via a symlink to the main repository- see the video below

Jira: https://codedotorg.atlassian.net/browse/SL-1548

Before: not much to video because the buttons weren't tab navigable at all

After:

Screen.Recording.2026-01-28.at.12.55.50.PM.mov

Followup: Make sure the student can then cycle through all the fish or trash floating in the pond to see how they did

<button
type="button"
onClick={this.toggleRecall}
aria-label="Toggle Identified Fish"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The pond doesn't always show Fish vs. Trash; it varies by level. We should reuse the wording from the two buttons in the interactive training phase, which in the first level is "Fish" vs. "Not fish".

It might be easier to label this something like "Show matching items" vs. "Show non-matching items".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants