- ๐ฎ๐ณIndia rinku jacob 13 Kerala
I have tested the patch with drupal version 9.5.3. Patch doesn't applied. getting some errors like this
Checking patch css/admin.toolbar.css...
error: while searching for:
border-right: 1px solid #ddd;
}.toolbar .toolbar-tray-horizontal ul ul li.menu-item:first-child {
border-top: 1px solid #ddd;
}.toolbar-tray-horizontal li.menu-item--expanded.hover-intent ul ul,
error: patch failed: css/admin.toolbar.css:34
error: css/admin.toolbar.css: patch does not apply
Checking patch js/admin_toolbar.hoverintent.js...
error: while searching for:
// Other wise we get unwanted behaviour where menu items are expanded while already in hovering other ones.
$(this).parent().find('li').removeClass('hover-intent');
$(this).addClass('hover-intent');
},
out: function () {
$(this).removeClass('hover-intent');
},
timeout: 250error: patch failed: js/admin_toolbar.hoverintent.js:6
error: js/admin_toolbar.hoverintent.js: patch does not apply - ๐ฆ๐ทArgentina abelpzl
#42 is applied correctly in 9.5.3 and in my opinion works as expected.
Thanks for providing this solution.
- ๐ฎ๐ณIndia sonam.chaturvedi Pune
Tried testing #42 with v3.3.0 on Drupal 10.1.x-dev.
On applying patch, getting below:Skipped patch 'css/admin.toolbar.css'. Skipped patch 'js/admin_toolbar.hoverintent.js'.
- last update
almost 2 years ago 17 pass - ๐บ๐ฆUkraine vselivanov Kyiv, Ukraine
Rerolled #40 for the v3.3.1 - 3.x-dev 50ff7eb6 commit. Tested with Drupal 9.5.8.
- Status changed to Needs work
almost 2 years ago 12:18pm 5 May 2023 - ๐ฎ๐ณIndia nikhil_110
Tried to apply patch #46 it is working fine for add content menu but not working properly for last content type in content type list for content type I have shared video.
need to work on it. - ๐ซ๐ทFrance tostinni
We've changed a little bit the patch so it can apply to Claro but also to Gin and works a little better with submenus as shown in #47 video.
We haven't tested yet on RTL theme and also the vertical admin menu bar for Gin is not functional, so keeping status as Need work.
- ๐ฒ๐ผMalawi chancenyasulu
patch #46 works for me in drupal 9.4.10 Panopoly thank you.
- last update
over 1 year ago 17 pass - ๐ฆ๐ทArgentina abelpzl
Patch #46 could not be applied in admin_toolbar 3.4.1 and drupal 9.5.10.
Instead #48 was applied correctly and seems to work correctly. - ๐จ๐ฆCanada darkodev
#48 works well for us on Drupal 10.1.3 and Admin Toolbar 3.4.1
- ๐ฎ๐ณIndia Indranil Roy kolkata
Indranil Roy โ made their first commit to this issueโs fork.
I have just made a couple of theming changes based on #48 to improve the appearance. Using Claro theme and Drupal 9.5.11.
- ๐บ๐ฆUkraine vselivanov Kyiv, Ukraine
I rerolled patch #48 for Admin Toolbar 3.5.0 on Drupal 10.3.1
The attached patch does not build on the currently dominant approach to this issue (#54 at the moment), and allows the sub-menus to scroll (rather than expand laterally) as mentioned in the original request. It introduces a new override template with an added container div around the ul's to achieve the desired result. This required some adjustments to the CSS and Javascript, including a direct copy/tweak of some of the core toolbar module code.
This is my first contribution, so kindly let me know if I've missed any steps or approached the process incorrectly. Based on 3.5.1 running on core 10.3.6.
- ๐ฎ๐ณIndia Kanchan Bhogade
Hi
I've tested patch #55 on Drupal 10.3.x
The patch is applied successfully...The menu dropdown is scrollable after applied patch
Screenshots