This is literally a copy of Diff: Missing column headings in Revisions list, which says:

Diff adds to columns to the Revisions list of a node, which get populated with radio buttons. However, since there are no headings to the columns it is completely unclear for the user what these buttons are supposed to do.

The patch is very similar and is attached.

I wonder if the whole form can be re-factored to extend Drupal/diff/Form/RevisionOverviewForm instead of re-creating it?

CommentFileSizeAuthor
entity_diff_ui-missing-column-headers.patch1.35 KBeuk

Comments

euk created an issue. See original summary.

mingsong’s picture

mingsong
Location 🇦🇺
commented

Thanks for raising it and the patch.

Waiting for the community to test and confirm.

I wonder if the whole form can be re-factored to extend Drupal/diff/Form/RevisionOverviewForm instead of re-creating it?

Good question, I think it depends on whether this module need to follow up the changes made for the Diff module in the feature. At this stage, I don't think so.

mingsong’s picture

mingsong
Location 🇦🇺
commented
Status: Active » Needs review