Updated: Comment 0

Problem/Motivation

BLA

Proposed resolution

BLUB

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#1 field_ui-2094097-1.patch10.73 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Active » Needs review
FileSize
field_ui-2094097-1.patch10.73 KB

This is just a starter

Status: Needs review » Needs work

The last submitted patch, field_ui-2094097-1.patch, failed testing.

swentel’s picture

swentel CreditAttribution: swentel commented 
+++ b/core/modules/field_ui/lib/Drupal/field_ui/Derivative/FieldLocalTasks.php
@@ -0,0 +1,164 @@
+} ¶

Nitpick: space ...

Not sure exactly about the breadcrumb tests, I'll see whether I can look at it this weekend too.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

These tests will be solved automatically with #2070651: Introduce a path-based breadcrumb builder, remove the one that's based on {menu_links}. in, which relies on #2070651: Introduce a path-based breadcrumb builder, remove the one that's based on {menu_links}. being in.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs work » Closed (duplicate)

#2111823: Convert field_ui / Entity local tasks to YAML definitions