Problem/motivation

BlockBlock1.png

BlockBlock2.png

Proposal

1. Prefix block editing page title with "Configure" to reflect the action.
2. Remove repeated Block, Block, Block and Block.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

LOL. All these changes look great. Can we add "after" screenshots?

Also, it looks like the custom block configuration part might collide with #1871772: Convert custom blocks to content entities, so we should make sure that issue incorporates these changes too once this goes in.

Status: Needs review » Needs work

The last submitted patch, block-block-block-block-block-block.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
708 bytes
3.81 KB
105.16 KB

Fixed upgrade test that referred to the title field's label. Also adding after screenshot:

xjm’s picture

Status: Needs review » Reviewed & tested by the community

Yay.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Oh, yes please. :)

Committed and pushed to 8.x. Thanks!

Gábor Hojtsy’s picture

Woot :)

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