Missing column headings in Revisions list

Rebased the MR.

We are using this patch and editors found it an improvement of the diff screen - I note there hasn't been much discussion on this since #11 so not setting to RTBC, although I can say I have one client happy using this approach.

Patch containing the current state of the merge request.

Sorry, ignore the patch in #22. This is the correct patch containing the current state of the merge request.

@miro_dietiker: Perhaps the column labels should be "Compare revision A" and "Compare revision B" or "Compare from" and "Compare to".

The question of the compare button on long forms is addressed in ✨ Add "compare revisions" button on top for long form. Needs review .

PHPStan is failing.

Rebased. phpstan passes.

+1 to the latest patch.

I'm not a huge fan of these column headings, and as stated already in the comments the A/B labelling may not make sense in other languages. With that being said, I don't really have any good alternatives but don't want to commit something before we get more of a consensus.

What about using "Source" and "Target"?

I've been using Gitlabs compare revisions a lot - and I know the context and UI is different, but I feel like maybe the same label be adapted?

E.g left column = "Source", right column = "Target"

"Source" and "Target" then with something to explain it in the help text link:

"When comparing selected revisions the changes are shown as if the source revision was being updated to the target revision."

I have a few clients who often use this screen, I can ask them to review options if we can get a few other ideas in the mix.

@ericgsmith source and target definitely make more sense to me. I'd be happy with that.

Perhaps it should be "Compare" instead of "Select" to make it clear that these are used for comparisons. I have rebased and updated the labels along these lines.

For other languages, these are translatable strings, so it could be whatever is desired in that language.

See MR comments.

Rebased. All comment threads resolved.

Sorry should've mentioned this needs to go into 2.x.

Rebased

Hi @liam morland,

I have replicated the issue and applied the patch you provided, confirmed the issue is solved, however, would it be possible to display the radio buttons at the center of the column. Please see images for reference.

Thanks,
Jake

The uncentered radio buttons

Working on it.

Please review it once.

Thanks for the patch.

Those CSS selectors work to center the radio buttons, but they will change other tables too. It is probably needed to add .diff-responsive-table to the selectors.

NW for #40.

Updated css file but i did not find .diff-responsive-table class in table instead it is .diff-revisions.