Updated: Comment #0


In #2058321: Move the 'place block' UI into the block listing I used "left" and "right" to divide the block list form into columns, that should have been "primary" and "secondary".
Furthermore, I made the change at the Form API level, making needless changes to the form structure.

Proposed resolution

Revert the Form API changes and use a template file

Remaining tasks


User interface changes


API changes

The form structure will be back as it was, so this reverts a minor API change.

#2078601: Move the block placement browser to the left of table and collapse it

#1 block-2079761-1.patch12.57 KBtim.plunkett
PASSED: [[SimpleTest]]: [MySQL] 58,129 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


tim.plunkett’s picture

Status: Active » Needs review
12.57 KB
PASSED: [[SimpleTest]]: [MySQL] 58,129 pass(es). View

I split this out of #2078601: Move the block placement browser to the left of table and collapse it because that issue doesn't have consensus.
In the meantime, this blocks all of the other patches I have that try to change the block form.

Cottser’s picture

Status: Needs review » Reviewed & tested by the community

This looks pretty great to me, big improvement IMO. The repetition of .layout-region is not ideal but there is #2061863: Make two column node CSS reusable to resolve that.

tkoleary’s picture


Looks awesome!

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

tim.plunkett’s picture

Assigned: tim.plunkett » Unassigned

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