Repository navigation
Start page first screen, sidebar toggle, done look for reviewed changes, faster start - #11
Merged
Merged
Conversation
…ange - Start page: on screens from 768px wide and 640px high, the hero sits centred in the first screen with "How it works" at its bottom edge and an arrow on to the rest (tighter spacing on laptop heights). Phones keep the plain flow. - Desktop: the "List 20" button in the top bar shows and hides the docked list of changes, and the list header has a collapse button. Before, the list could only be hidden from the View menu. - The current change is marked in the selection blue (edge and a light tint) instead of grey, which read as "disabled"; the same in the list and the overview bar. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Accepted: the new wording reads as plain text with a green edge (the removed words stay quietly struck through on the old side); before, it turned grey and looked disabled. Rejected: a solid neutral edge instead of a dotted one, and the ✗ in the text colour. Only colours change. - Desktop: a sidebar icon at the right end of the top bar shows and hides the list of changes, instead of the "List 20" button and a second button in the list header. Tablets keep the "List" button for the sheet. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Both readers (about 1 MB of JavaScript) were compiled and run on every page load, before the start page could paint, although only PDF and Word imports need them. They are now embedded as non-executed text, like the PDF.js worker, and started on first use. The page stays one self-contained file and still works offline. Starting them takes about 70 ms on a fast laptop (several times that on a phone); that time is now off the page load. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Between 640 and about 750 px of window height the hero and "How it works" did not fit, so the arrow sat below the fold and the steps were cut off. Below 760 px the start page now stays a plain flow, as before. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Summary
Test plan
python3 scripts/build.pyandnode scripts/check.mjs(new check: both libraries embedded as non-executed text)🤖 Generated with Claude Code