Skip to content

ACCESSIBILITY.md update for transcript navigation and scrolling#3721

Merged
corinagum merged 3 commits intomicrosoft:masterfrom
compulim:doc-accessibility-transcript-navigation
Feb 18, 2021
Merged

ACCESSIBILITY.md update for transcript navigation and scrolling#3721
corinagum merged 3 commits intomicrosoft:masterfrom
compulim:doc-accessibility-transcript-navigation

Conversation

@compulim
Copy link
Copy Markdown
Contributor

Changelog Entry

(No changelog for doc update)

Description

Update ACCESSIBILITY.md:

  • Add new section for transcript navigation
  • Add new section for transcript scrolling via keyboard
  • Update the section for "New messages" button
  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Comment thread docs/ACCESSIBILITY.md Outdated
Copy link
Copy Markdown
Contributor

@corinagum corinagum left a comment

Choose a reason for hiding this comment

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

LGTM! approved pending comments - lmk if you have questions

@corinagum corinagum merged commit 26e7613 into microsoft:master Feb 18, 2021
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