Skip to content

[UEPR-477] Tips Library - Focus after item select#449

Open
kbangelov wants to merge 4 commits intoscratchfoundation:release/UEPR-297-accessibility-improvementsfrom
kbangelov:task/uepr-477-tips-library-return-focus
Open

[UEPR-477] Tips Library - Focus after item select#449
kbangelov wants to merge 4 commits intoscratchfoundation:release/UEPR-297-accessibility-improvementsfrom
kbangelov:task/uepr-477-tips-library-return-focus

Conversation

@kbangelov
Copy link
Contributor

Resolves

https://scratchfoundation.atlassian.net/browse/UEPR-477

Proposed Changes

Made tutorials get focused after being selected. The focus return logic for the tutorials button was pretty much already there.

Reason for Changes

Part of accessibility initiative for Scratch.

@kbangelov kbangelov requested a review from a team as a code owner February 17, 2026 13:56
@kbangelov kbangelov changed the base branch from develop to release/UEPR-297-accessibility-improvements February 17, 2026 13:56
Copy link
Contributor

Choose a reason for hiding this comment

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

What feels a bit odd in the current implementation is that if the tutorial starts with anything other than a video, there's nothing actionable on the card. So even if it's focused, pressing Tab just takes you back to the beginning.

It might make sense to only make the card focusable when there's actually a video to play. I can't think of any other actionable elements on it right now. Ideally, users should also be able to navigate between steps or close the tutorial modal via keyboard, but that's outside the scope of this task. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So just without the image then? I agree.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments