Repository navigation
Annotated slide walkthrough on GitHub Pages, plus three fixes it found - #101
Merged
Merged
Conversation
- Quiz table: show the "log in to answer" prompt after logout. The coordinator's reset removed the logout listener before it could show the notice. - Scores modal and CSV export: skip the empty placeholder records that pad skipped questions, so they no longer appear as answers. - Login form: messages use border-box sizing so they do not overflow the panel. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019erPampkiqiri3Gab6oSHB
docs/walkthrough/ is a self-contained slide deck (no external libraries or fonts) that walks through the student and instructor journeys with 20 annotated screenshots and a sample CSV export. - scripts/capture-walkthrough.mjs (npm run walkthrough:capture) drives dita-demo with Playwright and records the screenshots plus the annotation boxes for each highlighted element. - pages.yml publishes it at /main/walkthrough/; pr-preview.yml at /pr-<n>/walkthrough/, and the preview comment links both. - ESLint ignores the generated img/shots.js. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019erPampkiqiri3Gab6oSHB
Contributor
🚀 PR Preview Deployment✅ Your changes are being built and deployed to a preview environment. 📋 Live PR Preview Links: ℹ️ About PR Previews:
📦 Production Demo (main branch): 🧪 Testing Locally: npm install
npm run build
# Open demo/quiz-index.html in your browser (use file:// or http-server)This comment will be automatically updated when you push new commits. |
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.
Follow-up to #100.
Walkthrough
docs/walkthrough/is a slide deck that walks through the student and instructor journeys. It has 25 slides, 20 annotated screenshots and a sample CSV export.#nlinks.Where it is published:
main:https://DeepBlueCLtd.github.io/BrowserTest/main/walkthrough/(added topages.yml)/pr-<n>/walkthrough/(added topr-preview.yml; the preview comment links both)To regenerate the screenshots, run
npm run build:dita:plain && npm run walkthrough:capture, thengit checkout -- dita-demo.scripts/capture-walkthrough.mjsdrivesdita-demo/with Playwright and records where each highlighted element sits. This is documented indocs/MAINTAINERS.md.The deck adds about 1.6 MB of PNGs to the repo.
Known gap: the demo output still has the search box. The capture hides it so the screenshots match the new template, and the intro slide says so.
Fixes found while capturing (tests first)
box-sizing: border-box.Verification
The encrypted-storage E2E mode was not re-run locally; CI runs it.
Generated by Claude Code