Problem/Motivation

This issue is part of #1875252: [META] Make the block plugin UI shippable. One of the most confusing aspects of the "interim" blocks user interface is that the UI does a poor job of explaining the workflow to users when they want to place additional blocks in their themes. This is the workflow currently:

First screen

screen_1.png

Second screen

screen_2.png

Two things are confusing:

  1. I clicked "Add block," and then I'm faced with two different kinds of block-adding tasks: placing a new instance of an existing block, or creating a new custom block and then additionally placing an instance of it.
  2. I am clicking the action "Add" and then being given the option to "configure" what look to be existing things, which doesn't make sense. In fact, there's also a bunch of configure buttons next to blocks on that first screen where I first selected "Add block"--it almost seems like I'm stuck in a loop.

Proposed resolution

Change the button text to describe the ACTUAL action we're taking: placing blocks. Both Bojhan and Kevin O'Leary seem to agree this would be a better button label.

  • The "Add block" button on admin/structure/block is re-labeled "Place blocks."
  • The "Configure block" buttons on admin/structure/block/list/block_plugin_ui%3Aythemename/add are re-labeled "Place block."

Remaining tasks

  • @klonos suggests removing the + icon in #4. @xjm still feels it makes sense since we are placing an additional block (added onto the ones already listed). Decide.

User interface changes

Before patch: screen 1

before_screen_1.png

Before patch: screen 2

before_screen_2.png

After patch: screen 1

after_screen_1.png

After patch: screen 2

after_screen_2.png

CommentFileSizeAuthor
#6 block-1950700-6.patch1.96 KBxjm
#6 interdiff-0.txt1.06 KBxjm
place-block.patch918 bytesxjm
after_screen_2.png17.98 KBxjm
after_screen_1.png57.85 KBxjm
before_screen_2.png19.64 KBxjm
before_screen_1.png55.79 KBxjm
screen_2.png66.89 KBxjm
screen_1.png33.82 KBxjm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Status: Active » Needs review

Status: Needs review » Needs work

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

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented

...coming from #1875780: "Configure block" button text in the Block Library is confusing that was just closed as a duplicate of this one here.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented

The "+" sign in from of the now renamed "Place blocks" button doesn't make sense in this new design. It should be removed.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented

The "+" sign in from of the now renamed "Place blocks" button doesn't make sense in this new design. It should be removed.

I think it still makes sense. It's also part of an existing pattern, not something special for this interface.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Status: Needs work » Needs review
FileSize
interdiff-0.txt1.06 KB
block-1950700-6.patch1.96 KB

I'll add the feedback from #4 to the summary. Attached updates the one test that relied on these link texts.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Issue summary: View changes

.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented

Thanx ;)

YesCT’s picture

YesCT CreditAttribution: YesCT commented

#1956346: 'Configure block' dropbutton link should say 'Add block' or 'Place block' for block library listings marked a duplicate of this one.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented

#6: block-1950700-6.patch queued for re-testing.

Bojhan’s picture

Bojhan CreditAttribution: Bojhan commented
Status: Needs review » Reviewed & tested by the community

I think this is a go, the only reservation I have is that we generally don't use action buttons to go to another listing page. This might end up confusing, but given that it is such a tiny change we can easily test it and adapt afterwards.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Status: Reviewed & tested by the community » Needs review

Related: #1956134: Provide helpful editing links on "admin/structure/block" for deriver blocks (menu, views, block content, etc.)

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Status: Needs review » Reviewed & tested by the community

Sorry, xpost. Thanks @Bojhan.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Yeah, I agree that this is totally confusing right now. This helps push it along a little towards sanity.

Committed and pushed to 8.x. Thanks!

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Updated issue summary.