Problem/Motivation

Many Drupal forms use an actions wrapper to display form actions, for example 'Submit'. The default administration theme also has styling specifically for the actions wrapper.

Proposed resolution

Use an actions wrapper to display form actions in the Term Merge forms.

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

The Term Merge forms now use an actions wrapper for a consistent form interface.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 term_merge-actions_wrapper-2980666-2.patch4.39 KBidebr
#2 term_merge-actions-wrapper-after.png35.24 KBidebr
#2 term_merge-actions-wrapper-before.png35.18 KBidebr

Comments

idebr created an issue. See original summary.

idebr’s picture

idebr commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new term_merge-actions-wrapper-before.png35.18 KB
new term_merge-actions-wrapper-after.png35.24 KB
new term_merge-actions_wrapper-2980666-2.patch4.39 KB

Attached patch implements an actions wrapper to the form actions in the Term Merge forms for a consistent form interface.

Before

After

wilfred waltman’s picture

wilfred waltman commented

I think @idebr is right.
Keep things consistent.

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me, thanks!

  • legolasbo committed 8c532f8 on 8.x-1.x authored by idebr 
    Issue #2980666 by idebr, legolasbo: Use an actions wrapper in the Term...
legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x. Thanks!

Status: Fixed » Closed (fixed)

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