Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Updated: Comment #0
Problem/Motivation
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
N/A
User interface changes
N/A
API changes
The form structure will be back as it was, so this reverts a minor API change.
Related Issues
#2078601: Move the block placement browser to the left of table and collapse it
Comment | File | Size | Author |
---|---|---|---|
#1 | block-2079761-1.patch | 12.57 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettI 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.
Comment #2
star-szrThis 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.
Comment #3
tkoleary CreditAttribution: tkoleary commented@tim.plunkett
Looks awesome!
Comment #4
Dries CreditAttribution: Dries commentedCommitted to 8.x. Thanks.
Comment #5
tim.plunkett