Problem/Motivation

The settings form is long and the changes in #3556528: Add drush command for content change report will make it even longer. Tighten up the existing settings into vertical tabs.

Issue fork revision_extras-3557900

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jastraat created an issue. See original summary.

jastraat opened merge request !7

jastraat’s picture

jastraat commented
Status: Active » Needs review
adriancooke’s picture

adriancooke commented

Suggestions

Add above accordion
<h2>Revision log form appearance</h2>

Edit accordion label
<summary>Customize form appearance</summary>

Edit heading
“Entity Types with Required Revisions”
(I would use sentence case and only capitalize Entity.)

Edit Content tab copy
“Content types that require new revisions and revision log messages.”
This is confusing to me. Could it say “Content types that require users to enter log messages.” ?

“Require the revision log message on the "add" forms.”
Could it say “Require messages when creating new content.” ?

“Applies to all the above content types that require revision log messages.”
Perhaps “Applies to all selected types on the ‘add’ form.” ?

Only local images are allowed.

Omit second sentence at top of accordion
After “Customizations apply…” delete “Leave blank…” as the information also appears below.

Edit appearance accordion copy
“Display the last revision log message below the log input/field.” ?

Only local images are allowed.

adriancooke’s picture

adriancooke commented
Status: Needs review » Reviewed & tested by the community

@jastraat the updated copy looks good to me. 👍 (Thanks for clarifying about requiring revisions vs. messages.) Changing status to reviewed.

  • jastraat committed bfe79e99 on 1.0.x 
    Issue #3557900: Improve settings form appearance
jastraat’s picture

jastraat commented
Status: Reviewed & tested by the community » Fixed

Thanks @adriancooke!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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