From 598cf4740f7069c56a884d4f80c437b95e565a3c Mon Sep 17 00:00:00 2001 From: Lars Eggert Date: Sat, 22 Jul 2023 15:03:36 -0700 Subject: [PATCH 1/2] fix: Fix menu hover issue Fixes #5702 --- ietf/static/css/ietf.scss | 17 +++++++++++++++++ ietf/static/js/ietf.js | 15 --------------- ietf/templates/base/menu.html | 1 - 3 files changed, 17 insertions(+), 16 deletions(-) diff --git a/ietf/static/css/ietf.scss b/ietf/static/css/ietf.scss index f705bc1cb1..4b01fbafec 100644 --- a/ietf/static/css/ietf.scss +++ b/ietf/static/css/ietf.scss @@ -84,6 +84,23 @@ html { scroll-padding-top: 60px; } +// Make submenus open on hover. +@include media-breakpoint-up(lg) { + :is(.dropdown-menu, .leftmenu .nav)>li>ul { + display: none; + } + + :is(.dropdown-menu, .leftmenu .nav)>li:hover>ul { + display: block; + } + + :is(.dropdown-menu, .leftmenu .nav) .dropdown-menu { + top: 0; + left: 100%; + right: auto; + } +} + // Make textareas in forms use a monospace font textarea.form-control { font-family: $font-family-code; diff --git a/ietf/static/js/ietf.js b/ietf/static/js/ietf.js index 75cf1e359c..69c8fa9ae6 100644 --- a/ietf/static/js/ietf.js +++ b/ietf/static/js/ietf.js @@ -93,18 +93,6 @@ $(document) $(document) .ready(function () { - - function dropdown_hover(e) { - var navbar = $(this) - .closest(".navbar"); - if (navbar.length === 0 || navbar.find(".navbar-toggler") - .is(":hidden")) { - $(this) - .children(".dropdown-toggle") - .dropdown(e.type == "mouseenter" ? "show" : "hide"); - } - } - // load data for the menu $.ajax({ url: $(document.body) @@ -140,9 +128,6 @@ $(document) } attachTo.append(menu.join("")); } - - $("ul.nav li.dropdown, ul.nav li.dropend") - .on("mouseenter mouseleave", dropdown_hover); } }); }); diff --git a/ietf/templates/base/menu.html b/ietf/templates/base/menu.html index d979808059..2f88aadca8 100644 --- a/ietf/templates/base/menu.html +++ b/ietf/templates/base/menu.html @@ -10,7 +10,6 @@