Problem/Motivation

From #2834253: Missing column headings in Revisions list, the button "Compare" at the end of the form is self-explaining. We could try to improve it to "Compare selection".

Proposed resolution

Change tests and value of the button.

Remaining tasks

User interface changes

API changes

Data model changes

Comments

Ginovski created an issue. See original summary.

ginovski’s picture

Status: Active » Needs review
StatusFileSize
new14.57 KB

Renamed to Compare selection.

miro_dietiker’s picture

Status: Needs review » Needs work

Admin content at admin/content uses the button label "Apply to selected items".
Funnily, admin/content/comment uses a completely different representation for bulk operations...
A last try to search for similarities: admin/modules shows "Install" on the button, to install the selected modules.

This issue proposes a 4th pattern that differs from all.
Just "Compare" as it is not is at least similar to the install page.
If like content, then "Compare selected items" or i'd prefer to adapt it to "Compare selected revisions".
Yes i'm also not a fan of spending a word more, but it seems we need to tell what we are comparing.

One thing though from though that caught my eye is: Our Compare button (whatever its label is) should be the primary operation, being a blue button!

ginovski’s picture

Status: Needs work » Needs review
StatusFileSize
new15.48 KB

1. Changed button label to 'Compare selected revisions' and set it as primary.
2. Changed tests accordingly.

There might be some fails in DiffViewModeTest, I think they are unrelated to this issue.
Edit: They were only local fails.

ginovski’s picture

StatusFileSize
new28.07 KB

Current form with the primary button screenshot:

johnchque’s picture

Status: Needs review » Reviewed & tested by the community

Looks good!

  • miro_dietiker committed a1767f0 on 8.x-1.x authored by Ginovski
    Issue #2838915 by Ginovski: Improve compare button to "Compare selection...
miro_dietiker’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thx.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.