Submenu Title [#2079715]

in custom.js, around line 94 you have:

var CloseItemInsert = '<li class="leaf close-item"><a href="#">submenu</a></li>';

Not sure if this is a bug, or an oversight, or something needed? I manually changed it to:

var CloseItemInsert = '<li class="leaf close-item"></li>';

This appears to work the same, except doesn't show the "submenu" title under a submenu. Not really sure why you need to label a submenu "submenu" if it's sort of assumed by having clicked on a menu item.

Comments

Comment #1

zymphonies-dev CreditAttribution: zymphonies-dev commented 13 September 2013 at 05:56

Hello,

Might be you talking about different theme because in Lexi Theme custom.js has 23 line code but you mentioned 94 line.

Anyway might be your drop down issue related with configuration not code issue. please check the link below.
http://www.zymphonies.com/blog/how-create-dropdown-menus-drupal

Thanks,
Shanid

Comment #2

zymphonies-dev CreditAttribution: zymphonies-dev commented 16 September 2013 at 07:03

Status:

Active

» Closed (fixed)

Hello,

Please download 7.x-2.x-dev version and check.

Thanks,
Shanid

Comment #3

WebWalker3D CreditAttribution: WebWalker3D commented 4 October 2013 at 16:23

Status:

Closed (fixed)

» Needs review

No, I am very much talking about your theme. There are 403 lines of code in 7.x-2.x-dev in custom.js

In the dev you have now, STILL, on line 95 it shows:

 var CloseItemInsert = '<li class="leaf close-item"><a href="#">submenu</a></li>';

I'd be more than glad to link the entire file if I really need to.

Comment #4

zymphonies-dev CreditAttribution: zymphonies-dev commented 7 October 2013 at 11:17

Version:	7.x-2.x-dev	» 7.x-1.1
Status:	Needs review	» Fixed

Hello,

You can download new release 7.x-1.1

http://ftp.drupal.org/files/projects/lexi-responsive-theme-7.x-1.1.zip

Thanks,
Shanid

Comment #5

WebWalker3D CreditAttribution: WebWalker3D commented 17 October 2013 at 23:25

Version:	7.x-1.1	» 7.x-2.x-dev
Priority:	Normal	» Major
Status:	Fixed	» Needs review

Hello,

I don't think you're quite understanding what I'm saying... First off, I'm not using 7.x.1.1 I am using 2.x-dev. And I'll state it a third time, in the 2.x-dev version, you have a bug in the code. If you are unwilling to fix it, fine - say so, but please do not tell me it's not there, and then refer this to a 1.x problem when I clearly stated it was a 2.x-dev issue. The status "closed (won't fix)" is there if you honestly can't be bothered.

Comment #6

zymphonies-dev CreditAttribution: zymphonies-dev commented 21 October 2013 at 13:31

Status:

Needs review

» Active

Hello,

This issue is fixed in 7.x-1.1 version.
you can use 7.x-1.1 version or you can replace custom.js script with below code.


jQuery(document).ready(function($) {
  $('.nav-toggle').click(function() {
    $('#main-menu div ul:first-child').slideToggle(250);
    return false;
  });
  if( ($(window).width() > 640) || ($(document).width() > 640) ) {
      $('#main-menu li').mouseenter(function() {
        $(this).children('ul').css('display', 'none').stop(true, true).slideToggle(250).css('display', 'block').children('ul').css('display', 'none');
      });
      $('#main-menu li').mouseleave(function() {
        $(this).children('ul').stop(true, true).fadeOut(250).css('display', 'block');
      })
        } else {
    $('#main-menu li').each(function() {
      if($(this).children('ul').length)
        $(this).append('<span class="drop-down-toggle"><span class="drop-down-arrow"></span></span>');
    });
    $('.drop-down-toggle').click(function() {
      $(this).parent().children('ul').slideToggle(250);
    });
  }
 
});

Thanks,
Shanid