Skip to content

Add UI navigation plain language rule to markdown accessibility instructions#1184

Open
v-rperez030 wants to merge 1 commit intogithub:stagedfrom
v-rperez030:add-ui-navigation-plain-language-rule
Open

Add UI navigation plain language rule to markdown accessibility instructions#1184
v-rperez030 wants to merge 1 commit intogithub:stagedfrom
v-rperez030:add-ui-navigation-plain-language-rule

Conversation

@v-rperez030
Copy link
Copy Markdown
Contributor

Adds guidance on writing UI navigation steps in sequential plain language rather than relying on icon names or visual breadcrumb notation.

Two new bullets added to the Plain Language section of instructions/markdown-accessibility.instructions.md:

  • Write actions as sequential steps in plain language first, using generic, stable labels rather than icon names or visual descriptions.
  • A parenthetical visual reference may follow as supplemental context, but never as the sole way to describe a navigation path.

This mirrors the rule recently merged in github/accessibility-governance#3229.

Copilot AI review requested due to automatic review settings March 26, 2026 18:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Markdown accessibility review instructions to encourage plain-language, step-by-step UI navigation guidance that doesn’t rely on icons or visual breadcrumb notation, improving clarity for non-visual and cognitively diverse users.

Changes:

  • Add a rule to describe UI navigation as sequential plain-language steps using stable labels.
  • Add a rule allowing visual/icon references only as supplemental parentheticals (not the primary navigation description).

@v-rperez030 v-rperez030 changed the base branch from main to staged March 26, 2026 18:32
@v-rperez030 v-rperez030 requested a review from dvelton as a code owner March 26, 2026 18:32
@aaronpowell
Copy link
Copy Markdown
Contributor

It looks like you've incorrectly branched from the main branch not staged, and as a result all the materialised plugins are included in this PR.

You can attempt to fix this with a rebase:

git fetch origin staged
git rebase --onto origin/staged origin/main <branch name>
git push --force-with-lease

If that does not resolve it, you can run npm run plugin:clean which will delete the materialised plugins and you can commit that change.

@v-rperez030 v-rperez030 force-pushed the add-ui-navigation-plain-language-rule branch from e29a2b9 to ccaa4d0 Compare March 27, 2026 15:39
@v-rperez030
Copy link
Copy Markdown
Contributor Author

Thanks for catching that, @aaronpowell! I've rebased the branch onto staged -- the diff should now only include the intended change (1 file, 2 lines added).

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.

3 participants