NEW dropdown top megamenu #13609
NEW dropdown top megamenu #13609
Conversation
|
@dreggy take a look at https://github.com/atm-john/dolibarr/tree/dreggy_drop_down_menu |
|
I copied your edits. By the way I think that a little delay on the menu popup will avoid erraneous opening of the menu while moving the mouse, for example while you transition the mouse to url bar or bookmarks. |
| @@ -530,6 +530,8 @@ function print_eldy_menu($db, $atarget, $type_user, &$tabMenu, &$menu, $noout = | |||
| foreach ($menu->liste as $menuval) { | |||
| print_start_menu_entry($menuval['idsel'], $menuval['classname'], $menuval['enabled']); | |||
| print_text_menu_entry($menuval['titre'], $menuval['enabled'], (($menuval['url'] != '#' && !preg_match('/^(http:\/\/|https:\/\/)/i', $menuval['url'])) ? DOL_URL_ROOT:'').$menuval['url'], $menuval['id'], $menuval['idsel'], $menuval['classname'], ($menuval['target'] ? $menuval['target'] : $atarget)); | |||
| if (!$conf->dol_no_mouse_hover) | |||
There was a problem hiding this comment.
need this :
if (!$conf->dol_no_mouse_hover && ($conf->global->TOP_MENU_USE_DROP_DOWN_MENU || $conf->global->MAIN_FEATURES_LEVEL >= 2))
| @@ -593,10 +674,10 @@ function print_text_menu_entry($text, $showmode, $url, $id, $idsel, $classname, | |||
| global $langs; | |||
|
|
|||
| if ($showmode == 1) { | |||
| print '<a class="tmenuimage" tabindex="-1" href="'.$url.'"'.($atarget ? ' target="'.$atarget.'"' : '').' title="'.dol_escape_htmltag($text).'">'; | |||
| print '<a class="tmenuimage" tabindex="-1" href="'.$url.'"'.($atarget ? ' target="'.$atarget.'"' : '').' title="'./*dol_escape_htmltag($text).*/'">'; | |||
There was a problem hiding this comment.
replace default behavior only if megamenu is displayed
| print '<div class="'.$id.' '.$idsel.' topmenuimage"><span class="'.$id.' tmenuimage" id="mainmenuspan_'.$idsel.'"></span></div>'; | ||
| print '</a>'; | ||
| print '<a '.$classname.' id="mainmenua_'.$idsel.'" href="'.$url.'"'.($atarget ? ' target="'.$atarget.'"' : '').' title="'.dol_escape_htmltag($text).'">'; | ||
| print '<a '.$classname.' id="mainmenua_'.$idsel.'" href="'.$url.'"'.($atarget ? ' target="'.$atarget.'"' : '').' title="'./*dol_escape_htmltag($text).*/'">'; |
There was a problem hiding this comment.
idem replace default behavior only if megamenu is displayed
|
On the last PR you asked me about "quick access" this a dirty quick exemple :
|
|
Ok, I get it, thanks. |
|
for the version 12 of Dolibarr, I wanted to dev a theme class with a theme class controller to allow theme to specify its own configurations compatibility ( and a lot of other cool stuff) but I couldn't this time... I will do it for the v13 of dolibarr |
|
Ok I will wait for that. |
|
it is still up to date |
eldy
added
the
PR postponed
New dropdown top megamenu
From a previous pull request #11792
The megamenu actually work on css :hover, without javascript.
The background color of the menu is a darker color of the menu color, if you change the menu color also the background of the megamenu will change.
There is a small css animation on the menu popup, can be removed or tuned.
The menu will not render well on smaller screens (mobile), I plan to make a mobile menu after finishing the desktop version.