WordPress.org

Ready to get started?Download WordPress

Forums

Clean Retina
[resolved] Menu Title on Mobile Issue (8 posts)

  1. djsirny
    Member
    Posted 1 year ago #

    I have been noticing something weird on mobile version with menubar. The only text that shows is the last page out of all options in the menu, and then that page is unclickable - can't get to it. Not sure what the text ought to be on mobile, but current page would be nice. And obviously would be nice to land on the last page, especially since it is my contact page. Desktop version working great. Like the theme!

    site = http://mykoreanteacher.com/

    Thanks for the help!

  2. Sanjiv
    Member
    Posted 1 year ago #

    Hi,

    I checked in your website and it looks like some kind of bug. We have to examine it further and after fixing the bug, we will make
    new update available fixing the bug.
    For now you could just create custom menu with the same pages as menu which are now as your menu and select that menu as your main menu.
    This will also solve your problem for now.

    Regards,
    Team Horse

  3. djsirny
    Member
    Posted 1 year ago #

    Hi, I look forward to the update. I have some other sites too that have a lot of pages and really don't want to try and replicate additional menus. Also, have quite a few updates from time to time to so that would be more additional work keeping pages & menus straight. Still like theme, but really hope this gets fixed for mobile.
    Thanks!

  4. bayoufox
    Member
    Posted 1 year ago #

    Same problem here.

    Site works reasonably good on laptops and pcs, but same menu issue as above.

    http://www.cherylvernonfox.com/develop/

  5. mrdares
    Member
    Posted 1 year ago #

    hi,
    to fix this, open the .js file in : theme/clean-retina/library/js/tinynav.js
    locate the lines :

    // Default settings
        var settings = $.extend({
          'active' : 'current-menu-item', // String: Set the "active" class
          'header' : false // Boolean: Show header instead of the active item
        }, options);
    
        var counter = -1;
    
        return this.each(function () {
          // Used for namespacing
          i++;
    
          var $nav = $(this),
            // Namespacing
            namespace = 'tinynav',
            namespace_i = namespace + i,
            l_namespace_i = '.l_' + namespace_i,
            $select = $('<select/>').addClass(namespace + ' ' + namespace_i);
    
          if ($nav.is('ul,ol')) {
    
            if (settings.header) {
              $select.append(
                $('<option/>').text('Navigation')
              );
            }

    replace 'header' : false by 'header' : true
    and replace $('<option/>').text('Navigation') by $('<option/>').text('my_menu') to give a custom name to your menu..

  6. mrdares
    Member
    Posted 1 year ago #

  7. mrdares
    Member
    Posted 1 year ago #

    in fact it doesn't really solve the problem since we want the 'current-menu-item' to be displayed, yes?

  8. djsirny
    Member
    Posted 1 year ago #

    Hello mrdares - thanks for the fix! Using that certainly helps fix the navigation issue on mobile and at least has a generic "Navigation" text for the menu instead of last page. Since page title shows in body I don't really "need" to have current page there in the menu and am fine with it just saying navigation for mobile use. Seems easy enough and certainly fixes the issue on my end.
    Thanks a lot!!!

Topic Closed

This topic has been closed to new replies.

About this Theme

About this Topic

Tags

No tags yet.