Background

When the Sub Menu is expanded, there should be no radius on the bottom of the button on hover/focus state.

Steps to Reproduce

Click on the sub menu to see the hover / focused state.

Acceptance Criteria

There should be no bottom radius on the expanded sub menu.

Screenshots

How Drupal UI currently looks like in dev
image of submenu focus in dev

How Drupal UI currently looks like in design
image of submenu with hover as designed

Issue fork drupal-3440220

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

jwitkowski79 created an issue. See original summary.

ckrina’s picture

Project: Navigation » Drupal core
Version: 1.x-dev » 11.x-dev
Component: Code » navigation.module

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

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

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

kostyashupenko’s picture

Resolved CSS linting issues. Just by running `yarn lint:css --fix`.

Issue seems fixed. Here are 4 screens illustration outline rings for hover and focus states, for both collapsed and expanded states.

Test

Test

Test

Test

nod_’s picture

Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed 9665396 and pushed to 11.x. Thanks!

nod_’s picture

  • nod_ committed 96653961 on 11.x
    Issue #3440220 by kostyashupenko, adelansari: Remove bottom radius on...

  • catch committed 051b9bae on 10.3.x authored by nod_
    Issue #3440220 by kostyashupenko, adelansari: Remove bottom radius on...
catch’s picture

Status: Patch (to be ported) » Fixed

  • nod_ committed 96653961 on 11.0.x
    Issue #3440220 by kostyashupenko, adelansari: Remove bottom radius on...

Status: Fixed » Closed (fixed)

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