Problem/Motivation

As part of #3484575: Move the Edit button outside the more actions drop down, the edit button was declared as primary page action for node edit page.

If Content Moderation is enabled and, there is any available forward revision or the current entity, when visiting the latest revision page, the Edit page action should be primary as well.

Steps to reproduce

  • Install Umami and enable Navigation Top Bar module
  • Edit any existing article or basic page and create a draft version
  • Confirm that "Latest version" option is added to the page actions dropdown
  • Click on that option
  • Confirm that Edit button is displayed as a regular page action instead of a primary one.

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 edit-latest-version-after.png141.82 KBplopesc
edit-latest-version.png144.55 KBplopesc

Issue fork drupal-3501934

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

plopesc created an issue. See original summary.

plopesc opened merge request !10996

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new edit-latest-version-after.png141.82 KB

Created MR that address the reported bug.

Screenshot of the new behavior in Latest version page.

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Title: Navigation Top Bar should Edit button as the primary action when viewing a forward revision » Navigation Top Bar should display Edit button as the primary action when viewing a forward revision
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me and the test class would be useful to extend upon in #3501332: Display content moderation state in the Navigation Top Bar

oily made their first commit to this issue’s fork.

nod_ closed merge request !10996

  • nod_ committed 74bdd70d on 11.x 
    Issue #3501934 by plopesc, oily, berdir: Navigation Top Bar should...
nod_’s picture

nod_
Primary language French
Location Lille
commented
Status: Reviewed & tested by the community » Fixed

Committed 74bdd70 and pushed to 11.x. Thanks!

Status: Fixed » Closed (fixed)

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