Follow up for #1871772-108: Convert custom blocks to content entities

Problem/Motivation

+++ b/core/modules/block/custom_block/lib/Drupal/custom_block/CustomBlockFormController.php
@@ -0,0 +1,216 @@
+ $form['additional_settings'] = array(
+ '#type' => 'vertical_tabs',
Our new standard is to call the vertical tabs container 'advanced' instead of additional_settings. Let's adjust that in a follow-up.

Proposed resolution

Rename from 'additional_settings' to 'advanced'

Remaining tasks

Write a patch

CommentFileSizeAuthor
#2 custom-blcoks-vtabs-1919934.2.patch1.17 KBlarowlan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Priority: Minor » Normal
Status: Postponed » Active
Issue tags: -Entity system, -D8 upgrade path
larowlan’s picture

Status: Active » Needs review
FileSize
1.17 KB

Straight forward, nice way to wind up an eventful day

sun’s picture

Status: Needs review » Reviewed & tested by the community
catch’s picture

Status: Reviewed & tested by the community » Fixed

OK. Committed/pushed to 8.x.

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