Problem/Motivation

The Content Moderation module adds \Drupal\content_moderation\Plugin\Menu\EditTab which changes the "Edit" tab to "New draft" or "Edit draft" depending on the state of the entity. The problem is that "New draft" assumes you are editing the entity in order to create a new draft. You might be archiving it or doing a quick fix on a published piece of content.

Proposed resolution

Remove the plugin and leave the tab as "Edit" because that is what the user is doing. What state the edit will be is a later concern. I.e. the moment you press save - or select the state prior to saving.

Remaining tasks

User interface changes

Remove plugin from content moderation.

API changes

None, a plugin is not an API.

Data model changes

None

CommentFileSizeAuthor
#3 2866656-3.patch8.07 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott’s picture

Status: Active » Needs review
FileSize
8.07 KB
yoroy’s picture

We discussed this a bit in IRC: I noticed the same, mentally stumbling over the "Edit draft" label. I think it's ok to just keep this "Edit" at all times. Edit is what you come to do. Deciding on what state to save those changes to is a later concern.

timmillwood’s picture

Status: Needs review » Reviewed & tested by the community

As long as testbot agrees I'm fine with this.

  • Gábor Hojtsy committed 58f4380 on 8.3.x
    Issue #2866656 by alexpott, yoroy, timmillwood: Clicking on "New Draft"...

  • Gábor Hojtsy committed eee757d on 8.4.x
    Issue #2866656 by alexpott, yoroy, timmillwood: Clicking on "New Draft"...
Gábor Hojtsy’s picture

Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

We ran into the same problem on the UX meeting where the new draft screen allowed to delete content (and not just the draft but the whole content). So this was indeed a mistake and should be removed. Thanks all!

Status: Fixed » Closed (fixed)

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