From 7c8492ba43e1d652c0404cf7dd4af77dad9c3ef5 Mon Sep 17 00:00:00 2001 From: Nariman Jelveh Date: Thu, 30 May 2024 17:09:18 -0700 Subject: [PATCH] Fix the issue of checkmark's wrong color in ctx menu disabled items on hover --- src/UI/UIContextMenu.js | 6 +++--- src/css/style.css | 3 +++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/UI/UIContextMenu.js b/src/UI/UIContextMenu.js index ae98b792..9a1e8c4d 100644 --- a/src/UI/UIContextMenu.js +++ b/src/UI/UIContextMenu.js @@ -176,7 +176,7 @@ function UIContextMenu(options){ return false; }); - // This will hold the timer for the submenu delay + // This will hold the timer for the submenu delay: // There is a delay in opening the submenu, this is to make sure that if the mouse is // just passing over the item, the submenu doesn't open immediately. let submenu_delay_timer; @@ -254,7 +254,7 @@ function UIContextMenu(options){ } }); - // useful in cases such as where a menu item is over a window, this prevents from the mousedown event + // Useful in cases such as where a menu item is over a window, this prevents from the mousedown event // reaching the window underneath $(`#context-menu-${menu_id} > li:not(.context-menu-item-disabled)`).on('mousedown', function (e) { e.preventDefault(); @@ -262,7 +262,7 @@ function UIContextMenu(options){ return false; }) - //disable parent scroll + // Disable parent scroll if(options.parent_element){ $(options.parent_element).css('overflow', 'hidden'); $(options.parent_element).parent().addClass('children-have-open-contextmenu'); diff --git a/src/css/style.css b/src/css/style.css index 0b174d2b..2f2b638e 100644 --- a/src/css/style.css +++ b/src/css/style.css @@ -1616,6 +1616,9 @@ span.header-sort-icon img { .context-menu .context-menu-item-active .context-menu-item-icon-active { display: inline-block; +} + +.context-menu .context-menu-item-active:not(.context-menu-item-disabled) .context-menu-item-icon-active { color: white; }