Unified diff components breaks Sourcegraph in MR
Description
Recently there's been some effort at cleaning up our diff components. This includes moving them from a <table>
/<tr>
structure to a css grid based structure.
Unfortunately, this breaks Sourcegraph form being able to detect the code info, since it's hardcoded to look for parent tr
.
Proposal
Probably the fastest path forward is to assist Sourcegraph in updating their parser.
Crazy/not-crazy idea
This seems like a good opportunity for me to share this proposal #233999. As opposed to forcing third-party extensions to have to traverse our ever-changing DOM structure, we could provide a API which they could interface with and plugin custom behavior.
Edited by Paul Slaughter