Skip to content

Commit d173457

Browse files
brovattenclaude
andcommitted
docs: index the 10 additional format variants posted on the mockup PR
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 332e5e3 commit d173457

1 file changed

Lines changed: 23 additions & 5 deletions

File tree

docs/mockups/pr-comment-file-dropdowns.md

Lines changed: 23 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,11 @@ The data is already there: every component in `analysis.json` carries
66
`file_methods[]` (file paths + methods), which the diff logic uses internally but
77
never renders.
88

9-
This doc mocks up two candidate formats, both using collapsed-by-default
10-
`<details>` blocks below the legend so the diagram stays the hero. The same two
11-
variants are posted as comments on the mockup PR so they can be compared exactly
12-
as reviewers would see them. All component/file/method names below are invented.
9+
This doc details two candidate formats (A and B below); ten further variants
10+
(C–L) live as comments on the mockup PR (#26) so they can be compared exactly as
11+
reviewers would see them — an index is at the end of this doc. All
12+
component/file/method names are invented, and every variant uses the same toy
13+
data so the formats are directly comparable.
1314

1415
---
1516

@@ -188,7 +189,24 @@ See this architecture in your editor: [**Open in VS Code →**](https://marketpl
188189

189190
---
190191

191-
## Implementation notes (if either variant is adopted)
192+
## Index of all variants (posted as comments on PR #26)
193+
194+
| Variant | Format | Always-visible footprint |
195+
|:-:|:--|:--|
196+
| A | One dropdown per component → changed files | one summary line per component |
197+
| B | Component → file dropdowns → methods (nested) | one summary line per component |
198+
| C | Compact pipe table with `+3 ~2 −1` method deltas | one table row per component |
199+
| D | Single dropdown → native diff manifest with `@@` component hunks | one line |
200+
| E | Single dropdown → monospace file tree grouped by directory | one line |
201+
| F | Blast-radius-ordered review checklist (live checkboxes) | one line per step |
202+
| G | Question-grouped: `[!IMPORTANT]` alert for removals + two dropdowns | alert + two lines |
203+
| H | HTML cross-tab (`+/~/−` columns) with dropdowns inside cells | one table row per component |
204+
| I | Release-notes prose: Added / Changed / Removed sentences | one sentence per component |
205+
| J | One italic "dek" sentence above the diagram + single changelog dropdown | one sentence + one line |
206+
| K | Typographic ledger: one line per component, methods in small type | one line per component |
207+
| L | Narrative paragraph with GFM footnotes carrying the manifest | one paragraph |
208+
209+
## Implementation notes (if a variant is adopted)
192210

193211
- **List changed files only.** Intersect each component's `file_methods[]` paths
194212
with the PR's git diff. A component can own 40 files while the PR touched 2 —

0 commit comments

Comments
 (0)