Cross-revision patch page
It could be useful to make the patch view more cross-revision, rather than looking strictly at only one revision at a time
- reviews, discussions might get scattered across revision
- revision descriptions are important, especially but not only the 1st
- revision selection is pretty basic at this point
Proposed solution:
- revision descriptions as a thread could double as revision selector
- instead of listing reviews and discussion and changes per revision, we could list only reviews or discussion or changes, but (optionally) across all revisions. This couldn’t apply to changes, though.
Benefits:
- have more of a patch-story rather than artefacts listed
- much simpler page structure
Drawbacks:
- we don’t know how important/urgent it is to improve the current version
👉 Click dummy – select 1st or last revision
The threaded revision descriptions could also be added without changing the rest of the page’s sctructure. See further designs in Figma
How would the “Across all revisions” tab work? Would we show each revision header and then the discussion thread beneath it?
Yea exactly, entire discussion thread separated by revision markers.
Would that mean that we duplicate the revision bodies that we see at the top as well?