Repository navigation
Fix the share button name, the empty heading, and llms.txt - #2264
Merged
Merged
Conversation
On small screens the share button shows only an icon, and its text was `display: none`, so the button had no accessible name. The text is now visually hidden in place of removed. The `h1` held only a link that is hidden on small screens, so the heading was empty there. It now has visually hidden text. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
No file existed, so the single page app fallback answered `/llms.txt` with `index.html`. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
|
Contributor
Footnotes
|
The span goes back to `hidden` on small screens, and the button gets `aria-label="Share"`. The template comments are gone too. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
NullVoxPopuli
approved these changes
Sep 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes three of the four findings in the PageSpeed report for
/edit. Local Lighthouse: accessibility goes from 0.90 to 0.96, and agentic browsing from 0.33 to 0.67.Fixed
display: nonethere. The button now hasaria-label="Share", and the span is unchanged.h1was empty on small screens, because it held only a link withhidden md:inline-block. It now has visually hidden text: "Limber, a playground and REPL"./llms.txtdid not exist, so the page fallback answered withindex.html. The newpublic/llms.txthas an H1, a summary, and links to the editor, the docs pages, and the repository.Not fixed: the tabster elements
tabsterputs two<i tabindex="0" aria-hidden="true" data-tabster-dummy>elements in<body>. Both audits that still fail (aria-hidden-focusand the agentic accessibility tree check) point at them.I tried
createTabster(window, { controlTab: false }), which removes them, and both categories then score 1.0. I did not keep it, because it changes keyboard behavior. The format switcher (GJS, GMD, Glimdown) is one Tab stop today, with arrow keys inside. WithcontrolTab: false, each button becomes a Tab stop. Arrow key navigation in the Demo menu still worked.The finding is mostly a false positive: the elements move focus away at once, so a person never rests on them. A real fix belongs in
tabsteror inember-primitives.Check
lint:hbsandlint:typespass.🤖 Generated with Claude Code