Попытка добавить вторую кнопку переключения для моего меню javascript - PullRequest
Новая
       28

Попытка добавить вторую кнопку переключения для моего меню javascript

0 голосов
/ 07 апреля 2020

Я действительно тупой с javascript, но я очень стараюсь. Я надеюсь, что кто-то может помочь.

Я создаю тему WordPress, используя начальную тему "Подчеркивания" от Automatti c. Тема имеет отзывчивое меню javascript с кнопкой, которую пользователь должен щелкнуть, чтобы показать / скрыть меню.

Я хочу добавить ВТОРОЙ кнопку, которая может вызвать меню EXACT SAME. Я попытался скопировать часть кода в теме, однако функция развертывания / свертывания кнопки, похоже, зависит от идентификатора родительского контейнера. Поэтому я пытаюсь создать кнопку, которая НЕЗАВИСИМА от родителя (кнопка, которую я могу поместить в любой div). Обратите внимание, оно должно вызывать то же меню, только альтернативную кнопку за пределами родительского элемента.

Вот исходный код HTML: 

 <nav id="site-navigation">
 <button class="menu-toggle" aria-controls="primary-menu" aria-expanded="false"><?php esc_html_e( 'Primary Menu', 'mytheme' ); ?></button>
    <?php wp_nav_menu( array('theme_location' => 'menu-1', 'menu_id' => 'primary-menu') ); ?>
    </nav>

Вот оригинал Javascript: 

( function() {
    var container, button, menu, links, i, len;

    container = document.getElementById( 'site-navigation' );
    if ( ! container ) {
        return;
    }

    button = container.getElementsByTagName( 'button' )[0];
    if ( 'undefined' === typeof button ) {
        return;
    }

    menu = container.getElementsByTagName( 'ul' )[0];

    // Hide menu toggle button if menu is empty and return early.
    if ( 'undefined' === typeof menu ) {
        button.style.display = 'none';
        return;
    }

    menu.setAttribute( 'aria-expanded', 'false' );
    if ( -1 === menu.className.indexOf( 'nav-menu' ) ) {
        menu.className += ' nav-menu';
    }

    button.onclick = function() {
        if ( -1 !== container.className.indexOf( 'toggled' ) ) {
            container.className = container.className.replace( 'toggled', '' );
            button.setAttribute( 'aria-expanded', 'false' );
            menu.setAttribute( 'aria-expanded', 'false' );
        } else {
            container.className += ' toggled';
            button.setAttribute( 'aria-expanded', 'true' );
            menu.setAttribute( 'aria-expanded', 'true' );
        }
    };

    // Get all the link elements within the menu.
    links    = menu.getElementsByTagName( 'a' );

    // Each time a menu link is focused or blurred, toggle focus.
    for ( i = 0, len = links.length; i < len; i++ ) {
        links[i].addEventListener( 'focus', toggleFocus, true );
        links[i].addEventListener( 'blur', toggleFocus, true );
    }

    /**
     * Sets or removes .focus class on an element.
     */
    function toggleFocus() {
        var self = this;

        // Move up through the ancestors of the current link until we hit .nav-menu.
        while ( -1 === self.className.indexOf( 'nav-menu' ) ) {

            // On li elements toggle the class .focus.
            if ( 'li' === self.tagName.toLowerCase() ) {
                if ( -1 !== self.className.indexOf( 'focus' ) ) {
                    self.className = self.className.replace( ' focus', '' );
                } else {
                    self.className += ' focus';
                }
            }

            self = self.parentElement;
        }
    }

    /**
     * Toggles `focus` class to allow submenu access on tablets.
     */
    ( function( container ) {
        var touchStartFn, i,
            parentLink = container.querySelectorAll( '.menu-item-has-children > a, .page_item_has_children > a' );

        if ( 'ontouchstart' in window ) {
            touchStartFn = function( e ) {
                var menuItem = this.parentNode, i;

                if ( ! menuItem.classList.contains( 'focus' ) ) {
                    e.preventDefault();
                    for ( i = 0; i < menuItem.parentNode.children.length; ++i ) {
                        if ( menuItem === menuItem.parentNode.children[i] ) {
                            continue;
                        }
                        menuItem.parentNode.children[i].classList.remove( 'focus' );
                    }
                    menuItem.classList.add( 'focus' );
                } else {
                    menuItem.classList.remove( 'focus' );
                }
            };

            for ( i = 0; i < parentLink.length; ++i ) {
                parentLink[i].addEventListener( 'touchstart', touchStartFn, false );
            }
        }
    }( container ) );
} )();

Итак, я попытался создать дополнительную переменную под названием «newbutton», которая нацеливает кнопку по идентификатору вместо тега. Идентификатор новой кнопки - «site-nav-button» 

newbutton = document.getElementById( 'site-nav-button' )[0];
    if ( 'undefined' === typeof button ) {
        return;
    }

Затем я скопировал и исправил следующую функцию: 

newbutton.onclick = function() {
        if ( -1 !== container.className.indexOf( 'toggled' ) ) {
            container.className = container.className.replace( 'toggled', '' );
            newbutton.setAttribute( 'aria-expanded', 'false' );
            menu.setAttribute( 'aria-expanded', 'false' );
        } else {
            container.className += ' toggled';
            newbutton.setAttribute( 'aria-expanded', 'true' );
            menu.setAttribute( 'aria-expanded', 'true' );
        }
    };

Затем я скопировал кнопку в HTML и добавил новый идентификатор: 

<button id="site-nav-button" class="menu-toggle" aria-controls="primary-menu" aria-expanded="false"><?php esc_html_e( 'Primary Menu', 'mytheme' ); ?></button>

Я пытался поместить это в другое место моей темы, но это не сработало. Я супер тупой с javascript. Что я пропускаю или ошибаюсь?

1 Ответ

0 голосов
/ 07 апреля 2020

по указанному вами коду кажется, что вы не объявляете переменную "newbutton". Например: var newbutton = '';.

Я не с компьютером сейчас, но этот код кажется правильным (если уже объявлена ​​переменная).

Попробуйте console.log (newbutton), чтобы увидеть, получает ли этот var элемент

РЕДАКТИРОВАТЬ: Я сейчас за компьютером и сделал тест с вашим кодом.

Проблема в этой строке: 

newbutton = document.getElementById( 'site-nav-button' )[0];

Вы должны изменить на: 

newbutton = document.getElementById( 'site-nav-button' );

Относительно того, как вы объявляете переменную newbutton, это правильно.

Надеюсь, это поможет.

...