Problem/Motivation

The Navigation module derives menu blocks but it keeps the same category and label so it is hard in the admin UI to know which block if from which module.

Steps to reproduce

Enable the Navigation module.
Go on Structure > Block Layout and try to place a block

Proposed resolution

Change the category.

Remaining tasks

Provide MR
Review

User interface changes

Block from the Navigation module will have a dedicated category.

Issue fork drupal-3460257

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

Grimreaper created an issue. See original summary.

grimreaper’s picture

Issue summary: View changes

inverted the screenshots.

grimreaper’s picture

Assigned: grimreaper » Unassigned
Status: Active » Needs review
smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

I like it. Definitely makes things easier to find/determine

quietone’s picture

Category: Feature request » Bug report
Issue tags: +Usability

I read the IS and comments. This is changing the UI, so adding tag.

I applied the diff and it works as expected, after clearing caches. I see that the after image in the issue summary does not match what I see in testing. In my testing '(navigation)' is '(Navigation)'. The latter is consistent with the other usages in the block list. And because of the consistency I am not tagging for a UX review.

I think this is a bug because it affects the functionality of the site for admins and developers.

Leaving at RTBC

  • nod_ committed da245219 on 10.3.x
    Issue #3460257 by Grimreaper, smustgrave, quietone: Have a dedicated...

  • nod_ committed 8912d593 on 10.4.x
    Issue #3460257 by Grimreaper, smustgrave, quietone: Have a dedicated...

  • nod_ committed ae41c62d on 11.0.x
    Issue #3460257 by Grimreaper, smustgrave, quietone: Have a dedicated...

  • nod_ committed 17aee791 on 11.x
    Issue #3460257 by Grimreaper, smustgrave, quietone: Have a dedicated...
nod_’s picture

Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 17aee79159 to 11.x and ae41c62daf to 11.0.x and 8912d59338 to 10.4.x and da2452190d to 10.3.x. Thanks!

grimreaper’s picture

Hi,

Thanks for the merge!

To reply to comment 6, I took the screenshots with the POC I did to see the fix and to create the issue. Then when doing the MR, I checked for other block categories with multiple words in it, then changed to capitalize for consistency.

Status: Fixed » Closed (fixed)

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