Skip to content

Update: Align 'skip to transcript' styles with Media (fixes #36)#47

Open
swashbuck wants to merge 1 commit into
masterfrom
issue/36
Open

Update: Align 'skip to transcript' styles with Media (fixes #36)#47
swashbuck wants to merge 1 commit into
masterfrom
issue/36

Conversation

@swashbuck

Copy link
Copy Markdown
Contributor

Fixes #36

Update

Align the "skip to transcript" button styles with the latest Media component accessibility approach:

  • Replace the :not(:focus-visible) visually-hidden rules with the core .visually-hidden-focusable mixin.
  • Add margin-bottom: @item-margin so the button has spacing when focused and visible.

Testing

  1. Build a course with a YouTube component that has a transcript enabled.
  2. Using keyboard navigation, Tab until you reach the "Skip to transcript" button.
  3. Confirm the button stays hidden until focused, appears with correct spacing on focus, and moves focus to the transcript on activation.

Posted via collaboration with Claude Code

@kirsty-hames kirsty-hames left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works as expected thanks @swashbuck 👍

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

Projects

Status: Needs Reviewing

Development

Successfully merging this pull request may close these issues.

Update "skip to transcript" styles as per Media

3 participants