This is a fairly minor issue, which contains a few easy-win fixes from #2662354: [META] Design new UX for Panels IPE:

  1. Highlight the selected layout anywhere it appears - The current layout is indicated if you're browsing a Layout category that includes it.
  2. If a layout doesn’t provide configuration just submit the form - Done, unless a user has specifically gone into the "Current layout" category and clicked update, as they probably want to know that there isn't anything to update. You can check this out by switching between one and two column layouts.
  3. On hover, surround the entire block with dashed line and change cursor to indicate that the whole block is movable. - Done
  4. Minus does not mean remove - Done
  5. Add text to Edit and Save./Only show Save when something can be saved. - Done.

In addition, I've also fixed an annoying UX problem where when dragging a block you would see droppable regions that would not change the weight. We should only show droppable regions that actually matter, so I fixed that as well.

I've also hidden the "On screen" category if there are no blocks on screen.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

samuel.mortenson created an issue. See original summary.

japerry’s picture

Status: Needs review » Needs work

So after applying this, if I edit an existing block, it moves down to the bottom of the layout region. I didn't recall this happening for the patch.

Either way, that should be fixed in this issue.

samuel.mortenson’s picture

Status: Needs work » Needs review

While that is a bug, it's unrelated to this issue. Confirmed by un-applying the patch and testing the same thing. Filed #2665836: Moving to different regions, changing weight, and removing blocks should trigger an AJAX call in the IPE to address the larger issue/related confusion.

japerry’s picture

Status: Needs review » Fixed

Sweet, that makes sense. fixed!

Status: Fixed » Closed (fixed)

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