Repository navigation
Mobile: bring M25 screen reader and M19 full text into main - #77
Merged
Merged
Conversation
Spoken labels, roles and states for buttons (symbol buttons named: Back, Forward, Settings, zoom), Mosaic tiles, canvas cards, minimap, timeline items, keyword chips, results, tab rows and address suggestions. Sheet titles and timeline sessions are headings; Timeline/Canvas/Replay are tabs. Gesture-only actions get screen-reader actions (next/previous trail, hide keyword, close tab). The capture note is announced (VoiceOver) and is a polite live region (TalkBack). Only mobile/ changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X9wDmS5VvrWmyHhcWA9MoL
Pages with a DOI or PMID (found on the phone with the capture script,
nothing sent) show a Full text button. Tapping it looks the DOI up with
Unpaywall (the person's email) and OpenAlex (the person's free API key),
merges copies (PDFs and published versions first; version, licence,
where), and always offers LibKey ("Through your library") and doi.org.
Sven's decision (option a): each person adds their own email and key in
Settings, kept in the keychain. Settings and the sheet show a set-up
tutorial and a "What NODE sends" disclaimer; off by default. Requests go
through core/net (cached, rate limited); cache keys leave out email/key.
Only mobile/ changes.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01X9wDmS5VvrWmyHhcWA9MoL
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
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.
This brings the changes from #70 (M25 screen reader structure) and #72 (M19 full-text chip) into
main.Those two PRs were stacked: #70 targeted the branch of #69, and #72 targeted the branch of #70. GitHub merged each one into the branch below it, not into
main. Somainonly has #69 (smart capture). This branch is #72's branch with currentmainmerged in. There were no conflicts, and onlymobile/changes.Checks:
npm testinmobile/passes (16), and both bundles build.Generated by Claude Code