-
Notifications
You must be signed in to change notification settings - Fork 0
Show the form's own name on the forms source card, tidy badges #2041
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+32
−8
Merged
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
4dfe2b2
fix(forms): show the form's own name on the source card, tidy badges
claude 7c198a9
docs(ledger): record handoff review for the forms source-card relabel
claude a603e69
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 30d50c9
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 0517f2a
fix(forms): use formCode() for the code badge, pill rounding for stat…
claude 0e97aa6
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo d04d8f3
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo b508ad0
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 83d20a7
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo d875a3e
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo ce583e7
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 281f19e
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 96ed06f
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 932c9f9
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo b83726c
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 607e384
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo d5ce7f8
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 4bd3f8e
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo fafc039
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 2be452c
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo 31508b8
Merge branch 'main' into claude/form-names-design-rjw40t
BigSimmo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
1 change: 1 addition & 0 deletions
1
...ords/eac503b9ddf311057c2d0b0c05ab3cc7cb240a5eb34fbed37920b32612534694.record.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| | 2026-08-17 | claude/form-names-design-rjw40t | 4dfe2b229269ec55a6ebbfc9493317c2e05670fd | src/components/forms/form-detail-page.tsx | PR #2041 opened (BigSimmo/Database). User-requested UI relabel of the forms detail page's source-file card: dropped the synthetic 'Form {code}.pdf' heading (a made-up filename) in favour of the form's own title (form.title), moved the form code into a small kicker badge, and turned the 'Password protected'/'Check source' text into a proper tone-pill status badge on both breakpoints. Static MHA-2014 forms reference catalogue only; no form content, availability data, or source URLs changed. classifyPullRequestFiles: not clinicalRisk, not operationalRisk, not RAG-ranking, so no Clinical Governance Preflight or RAG impact line required. | Verification not run: session environment has Node 22 with no node_modules installed (repo requires Node >=24.15.0 <25, engine-strict); npm ci was not attempted (no install/network side effects requested). Reviewed diff by hand: balanced JSX tags/braces across the file (git diff + brace-count check), every Tailwind class/token used (text-3xs, text-2xs, tracking-label, toneWarning, toneNeutral) already defined/used elsewhere in the repo. Grepped tests/** for the changed copy ('Password protected', the .pdf-suffixed heading, formShortTitle) - no test pins it. PR body asks the merger to run npm run verify:pr-local and a quick Chromium/phone check of /forms/* before merge. | |
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.