Background

For grandchildren menu items, the gray menu box needs to have more padding on the bottom. There should be an additional 12px of padding along the bottom of the box. Currently, there isn’t any additional padding

Steps to Reproduce

Hover on a menu item that has children. Click on a child that has grandchildren. Note the bottom padding of the gray box that contains the grandchildren

Acceptance Criteria

The gray box that contains the grandchildren has an additional 12px of padding along the bottom

Links

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

Screenshots

What it currently looks like in dev
Image of gray container for grandchildren menu on dev

How it is designed
image of gray container for grandchildren menu as designed

CommentFileSizeAuthor
grandchildren-padding-design.png54.44 KBjwitkowski79
grandchildren-padding-onDev.png55.59 KBjwitkowski79

Issue fork navigation-3440228

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.

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

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

bronzehedwick opened merge request !245

bronzehedwick’s picture

bronzehedwick
he/him
Location New York
commented
Status: Active » Needs review
ckrina’s picture

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

Moving to Needs Work to use the design system variables instead of pixels.

bronzehedwick’s picture

bronzehedwick
he/him
Location New York
commented
Assigned: Unassigned » bronzehedwick
bronzehedwick’s picture

bronzehedwick
he/him
Location New York
commented

Thanks for catching @ckrina! I've updated the MR.

bronzehedwick’s picture

bronzehedwick
he/him
Location New York
commented
Assigned: bronzehedwick » Unassigned
Status: Needs work » Needs review
ckrina’s picture

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

Merged, thanks!

Status: Fixed » Closed (fixed)

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