Updated: Comment #0

Problem/Motivation

Steps:
1) visit admin/structure/display-modes/form - only User is listed
2) manually enter admin/structure/display-modes/form/add/node - now you can add new form mode for Node
3) the same applies for view modes admin/structure/display-modes/view

Proposed resolution

Create a local action for admin/structure/display-modes/[form|view]/add route that already works but not accesible from UI
Optinally we could get rid of links under each entity

Remaining tasks

create patch and write tests

User interface changes

New local action

API changes

no

CommentFileSizeAuthor
#2 form-mode-after.png11.18 KBandypost
#2 form-mode-before.png8.78 KBandypost
#1 2199875-add-mode-1.patch1.78 KBandypost
PASSED: [[SimpleTest]]: [MySQL] 64,547 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

andypost’s picture

Status: Active » Needs review
FileSize
1.78 KB
PASSED: [[SimpleTest]]: [MySQL] 64,547 pass(es). View

Patch with tests

andypost’s picture

FileSize
8.78 KB
11.18 KB

Screenshots:

before

after

tim.plunkett’s picture

Status: Needs review » Reviewed & tested by the community

Oh! These are still in entity_menu(), they must have just gotten missed during local action conversion.
I *knew* I'd written routes for them, and it didn't make sense why it wasn't showing up.

We should just leave the entity_menu() bits for #2177041: Remove all implementations of hook_menu.

alexpott’s picture

Don't the default form modes need to exist too?

So if you add a form mode for content you and select it to be customisable can can not also customise the default.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

@swentel took me through some of my confusion.

Patch works!

Committed ab2a911 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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