Problem/Motivation

test

Steps to reproduce

1. git checkout 1.x
2. git pull --rebase
3. If you recently already had `Navigation` module enabled on your local instance, then run drush pmu -y navigation && drush en -y navigation // easiest way, since a lot of was merged recently in 1.x
4. Create any node
5. Visit node page
6. Click on "More actions" in top bar and try to click in outside area.

Issue fork navigation-3415785

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

kostyashupenko created an issue. See original summary.

kostyashupenko’s picture

Issue summary: View changes
kostyashupenko’s picture

Issue tags: +Novice
kostyashupenko’s picture

Issue tags: +JS, +frontend
kostyashupenko’s picture

Issue summary: View changes
ahsannazir’s picture

Assigned: Unassigned » ahsannazir
Status: Active » Needs work

ahsannazir’s picture

Assigned: ahsannazir » Unassigned
Status: Needs work » Needs review
StatusFileSize
new322.14 KB

Raised MR for hiding dropdown when clicking outside dropdown

finnsky’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me! Thank you!

sandeep sanwale’s picture

StatusFileSize
new1.75 KB
new316.49 KB

Done some changes in dropdown JS.

finnsky’s picture

@Sandeep Sanwale thank you for your patch.
In this ticket it is enough to get just `close on outside click`

In your patch i see things which we need to discuss first. like:
`e.stopPropagation();`

Please feel free to open new issue with that javascript optimisations.

ckrina’s picture

Status: Reviewed & tested by the community » Fixed

Merged, thanks!
And for the JS improvements, feel free to open a new issue as @finnsky says so those specific suggestions can be discussed without blocking this :)

Status: Fixed » Closed (fixed)

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