Background

The color of the sub menu title is incorrect. The color of the title should be #13161A (Text - Extra Dark). On the test link, it’s showing var(--drupal-admin-color-gray-800,#4f5661)

Steps to Reproduce

Click on a menu item that has children. Inspect the title that appears at the top of the submenu

Acceptance Criteria

The color of the submenu title is updated to #13161A (Text - Extra Dark)

Links

Link to Figma design: https://www.figma.com/file/VCPAxetieAC2pzw7hgX3ij/Drupal-Admin-UI---Design-System?type=design&node-id=667%3A18270&mode=dev

Screenshots

What it looks like on dev
Image of submenu title on dev

What it looks like as designed
Image of submenu as designed

CommentFileSizeAuthor
#3 After-patch.png61.04 KBkanchan bhogade
#3 Before-patch.png46.97 KBkanchan bhogade
#2 navigation-3440226.patch2.51 KBrajan kumar@2026
submenu-title-design.png55.56 KBjwitkowski79
submenu-title-onDev.png30.14 KBjwitkowski79

Issue fork drupal-3440226

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

jwitkowski79 created an issue. See original summary.

rajan kumar@2026’s picture

rajan kumar@2026 commented
Status: Active » Needs review
StatusFileSize
new navigation-3440226.patch2.51 KB

hii @jwitkowski79

I Updated color of the title #13161A (Text - Extra Dark).

kanchan bhogade’s picture

kanchan bhogade commented
StatusFileSize
new Before-patch.png46.97 KB
new After-patch.png61.04 KB

Hi
I've tested the "navigation-3440226" patch on Drupal 11.x
The patch is applied cleanly...

Test Result:
The color of the sub-menu title is #13161A.
RTBC+1

Adding screenshots

ckrina’s picture

ckrina
she/her
Primary language Catalan
Location Barcelona
commented
Project: Navigation » Drupal core
Version: 1.x-dev » 11.x-dev
Component: Code » navigation.module
ckrina’s picture

ckrina
she/her
Primary language Catalan
Location Barcelona
commented
Status: Needs review » Needs work

Moving to needs work because this needs to be applied to core now.

Also, @Rajan Kumar@2026 we recommend that you convert this patch to a merge request. Merge requests are preferred over patches. Be sure to hide the old patch files as well.

sanisaha’s picture

sanisaha commented

I am currently working on this issue

sanisaha opened merge request !7789

sanisaha’s picture

sanisaha commented
Status: Needs work » Needs review

I have created a MR for this issue. Submenu title now have desired color code.

finnsky’s picture

finnsky commented
Status: Needs review » Needs work

Thank you for patch. I've added couple of things to fix.

sanisaha’s picture

sanisaha commented
Status: Needs work » Needs review

I have updated the MR with necessary changes

finnsky’s picture

finnsky commented
Status: Needs review » Needs work
finnsky’s picture

finnsky commented

Also probably no need to add quotes style here.

sanisaha’s picture

sanisaha commented
Status: Needs work » Needs review

updated the MR with requested changes

smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

Left some comments on the MR.

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

adwivedi008’s picture

adwivedi008 commented
Status: Needs work » Needs review

Reverted out-of-scope changes suggested by @smustgrave on #14

Setting the issue to Needs review

smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

Still 1

adwivedi008’s picture

adwivedi008 commented
Status: Needs work » Needs review

Reverted the remaining issue.

Setting the issue to Needs review

smustgrave’s picture

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

Feedback appears to be addressed.

  • nod_ committed dc7a4299 on 11.x 
    Issue #3440226 by sanisaha, adwivedi008, Rajan Kumar@2026, jwitkowski79...

  • nod_ committed 9def0ac7 on 10.3.x 
    Issue #3440226 by sanisaha, adwivedi008, Rajan Kumar@2026, jwitkowski79...

nod_ closed merge request !7789

nod_’s picture

nod_
Primary language French
Location Lille
commented
Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed dc7a429 and pushed to 11.x. Thanks!

  • nod_ committed dc7a4299 on 11.0.x 
    Issue #3440226 by sanisaha, adwivedi008, Rajan Kumar@2026, jwitkowski79...

Status: Fixed » Closed (fixed)

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