Contrib.social

Bottom padding needs to be increased along gray container box of grandchildren menu items

Open on Drupal.org →
Created on 11 April 2024, 10 months ago
Updated 29 April 2024, 9 months ago

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

How it is designed

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇺🇸United States jwitkowski79

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @jwitkowski79
  • First commit to issue fork.
  • First commit to issue fork.
  • Merge request !245Issue #3440228: Add extra padding below grandchild menu items → (Merged) created by bronzehedwick
  • Status changed to Needs review 10 months ago
  • Comment 10 months ago →
    🇺🇸United States bronzehedwick New York
  • Pipeline finished with Success
    10 months ago
    Total: 211s
    #145316
  • Status changed to Needs work 10 months ago
  • Comment 10 months ago →
    🇪🇸Spain ckrina Barcelona

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

  • Assigned to bronzehedwick
  • Comment 10 months ago →
    🇺🇸United States bronzehedwick New York
  • Comment 10 months ago →
    🇺🇸United States bronzehedwick New York

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

  • Issue was unassigned.
  • Status changed to Needs review 10 months ago
  • Comment 10 months ago →
    🇺🇸United States bronzehedwick New York
  • Pipeline finished with Success
    10 months ago
    Total: 284s
    #147157
  • Status changed to Fixed 10 months ago
  • Comment 10 months ago →
    🇪🇸Spain ckrina Barcelona

    Merged, thanks!

  • Pipeline finished with Success
    10 months ago
    Total: 226s
    #147208
  • Comment 10 months ago →
    System Message
  • Comment 9 months ago →
    System Message

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

  • Pipeline finished with Canceled
    9 months ago
    Total: 1s
    #163371
  • Pipeline finished with Skipped
    9 months ago
    #163376
  • Pipeline finished with Failed
    6 months ago
    Total: 51s
    #246589
  • Pipeline finished with Failed
    6 months ago
    Total: 53s
    #248028
  • Pipeline finished with Failed
    5 months ago
    Total: 515s
    #272354
  • Pipeline finished with Failed
    5 months ago
    Total: 553s
    #275697
  • Pipeline finished with Canceled
    2 months ago
    Total: 48s
    #356939
  • Pipeline finished with Canceled
    2 months ago
    Total: 102s
    #356941
  • Pipeline finished with Failed
    2 months ago
    Total: 105s
    #356954
  • Pipeline finished with Failed
    2 months ago
    Total: 753s
    #356969
  • Pipeline finished with Canceled
    2 months ago
    Total: 195s
    #357028
  • Pipeline finished with Success
    2 months ago
    #357031
  • Pipeline finished with Success
    about 1 month ago
    Total: 248s
    #380478
  • Pipeline finished with Success
    18 days ago
    Total: 149s
    #399624
contrib.social Blog FAQ Discussions
Production build 0.71.5 2024