Problem/Motivation

)$ git grep "'toolbar'" | grep -v core/modules/toolbar | grep -i help
core/modules/contextual/src/Hook/ContextualHooks.php: ':toolbar' => \Drupal::moduleHandler()->moduleExists('toolbar') ? Url::fromRoute('help.page', [
core/modules/shortcut/src/Hook/ShortcutHooks.php: ':toolbar-help' => \Drupal::moduleHandler()->moduleExists('toolbar') ? Url::fromRoute('help.page', [

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3560205

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:

Comments

quietone created an issue. See original summary.

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

dcam’s picture

Status: Active » Needs review

The contextual module toolbar button was omitted from Navigation without replacement. See #3465295: Integrate Top Bar Navigation with Contextual editing. So that help text was removed entirely.

smustgrave’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

Think changing to menu is fine. It's accurate just verified "Navigation user links" created with navigation.

  • catch committed fad3b98f on 11.x
    task: #3560205 Remove toolbar usage in hook help
    
    By: quietone
    By: dcam
    
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x, thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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