docs: clear wiki-checklist debt for /display and What's That Card updates - #309
Merged
Merged
Conversation
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
Clears the accumulated wiki-maintenance checklist items left by six recently-merged PRs, per CLAUDE.md's wiki-maintenance convention (
docs/user-guide.mdis the wiki+site-published guide; edits here regenerate theUser-Guidewiki page and/guide/using-it, never hand-edited directly).+ Add Cardsdropdown (verified the actual button label inImport.tsx— the PR's own body flags it as not literally "Import ▾" despite the mockup, so the guide describes what's actually on screen).docs/features/printing-tags.md's own Frontend section already reads correctly post-merge (PR Quiz-reveal hero redesign for What's That Card? (#305) #308 updated it directly), so only the user-guide needed this pass.PR #303 (sitewide dark-chrome/orange retheme): no doc/wiki change needed — confirmed no
docs/file describes the old blue accent as current (PR #303's own body already checked this; independently re-confirmed via grep for the old hex values and "blue" theme language).Verification
python3 .github/scripts/docs_lint.py— clean.npx prettier@2.7.1 --check docs/user-guide.md— clean.DisplayPage.tsx,FinishFooter.tsx,CardSpacingControl.tsx,SavedDecksLandingPanel.tsx,Import.tsx) and against each PR's own merged body, not just the task's framing — caught and fixed one drafting error this way (initially wrote "Import ▾" before confirming the rendered button actually reads "+ Add Cards").Open items / decisions needed
None — all six PRs' merge-time wiki-checklist items are addressed by this change.
Not merging this myself — leaving for review per repo convention.