Problem/Motivation

The FloodUnblockAdminForm button "Remove selected items from the flood table" is a form action, but this button is not wrapped in an actions wrapper. For consistency with bulk operations, for example on /admin/content, let's add an actions wrapper.

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

The button "Remove selected items from the flood table" on the FloodUnblockAdminForm has an actions wrapper for consistency with bulk actions

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 3313855-2.patch765 bytesidebr

Comments

idebr created an issue. See original summary.

idebr’s picture

idebr commented
Title: Adds actions wrapper to FloodUnblockAdminForm for consistency with bulk actions » Add actions wrapper to FloodUnblockAdminForm for consistency with bulk actions
Status: Active » Needs review
StatusFileSize
new 3313855-2.patch765 bytes

Attached patch adds an actions wrapper to FloodUnblockAdminForm for consistency with bulk actions.

batigolix’s picture

batigolix
Primary language Dutch
Location Utrecht
commented
Assigned: Unassigned » fabianderijk
Status: Needs review » Reviewed & tested by the community

Patch applies well. Action keeps working well

  • fabianderijk committed 1c8fc59 on 2.2.x 
    Issue #3313855 by idebr, batigolix: Add actions wrapper to...
fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
commented
Status: Reviewed & tested by the community » Fixed

The patch has been committed tot the 2.2.x branch

Status: Fixed » Closed (fixed)

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