Neat UX Trick Не работает. Как я могу это исправить? - PullRequest
Новая
       89

Neat UX Trick Не работает. Как я могу это исправить?

0 голосов
/ 20 июня 2020

Вопрос

Закомментированный код в верхней части javascript - это моя попытка сделать так, чтобы соответствующая выбранная вкладка кнопки (на панели навигации меняет цвет, получает фон и др. c). Но когда я раскомментирую код вверху, он не только не работает, но и гамбургер-меню тоже перестает работать!

Как мне настроить его, чтобы класс работал (чтобы выбранная вкладка стала очевидной пользователю)?

Мой код 

//Nav Bar!!

//Button selected gets color change

/*
const selectedNav = document.querySelectorAll('li');

selectedNav.forEach((item)=>{
    document.selectedNav.addEventListener('click', navChange)
})

function navChange(event){
    ul.forEach((item)=>{
        item.classList.remove('add-this-to-selected-section');
    })
    event.target.classList.add('add-this-to-selected-section');
}
*/

const navSlide = () =>{
    const burger = document.querySelector('.burger');
    const nav = document.querySelector('.nav-links');
    const navLinks = document.querySelectorAll('.nav-links li');

// Toggle Nav
    burger.addEventListener('click',()=>{
        nav.classList.toggle('nav-active');


    //Animate Links
    navLinks.forEach((link, index)=>{
        if(link.style.animation){
            link.style.animation = '';
        } else {
        link.style.animation = `navLinkFade 0.5s ease forwards ${index / 7 + 0.5}s`
        }
   
    })
    //Burger Animation

    burger.classList.toggle('toggle');

})

}

navSlide();
nav{
    display: flex;
    justify-content: space-around;
    align-items: center;
    min-height: 8vh;
    font-family: 'Poppins', sans-serif;
    background-color: #ffffff;
}


.nav-links{
    display: flex;
    justify-content: space-around;
    width: 50%;
}

.nav-links li{
   list-style: none;
}

.nav-links a{
    color: #245871;
    text-decoration: none;
    letter-spacing: 3px;
    font-weight: 800;
    font-size: 18px;
}

.add-this-to-selected-section{
    color: white;
    background-color: #245871;
    padding: 5px;
    border-radius: 5px;
}

.burger div{
    width: 25px;
    height: 3px;
    background-color: #245871;
    margin: 5px;
    transition: all 0.3s ease;
}

.burger{
    display: none;
    cursor: pointer;
}


@media screen and (max-width: 1024){
    .nav-links{
        width: 70%;
        cursor: pointer;
        font-weight: 1200;
    }
}

@media screen and (max-width: 997px){
   body{
       overflow-x: hidden;
   }
   
   nav{
       color: white;
   }

   .head-btn{
       color: white;
   }

    .nav-links{
        position: absolute;
        right: 0px;
        height: 92vh;
        top: 8vh;
        background-color: #245871;
        color: white;
        display: flex;
        flex-direction: column;
        align-items: center;
        width: 70%;
        transform: translateX(100%);
        transition: transform 0.5s ease-in;
    }

    .nav-links a{
        opacity: 1;
        color: white;
    }
    .burger{
      display: block;
    }
}

.nav-active{
    transform: translateX(0%);
    color: white;
}

@keyframes navLinkFade{
    from{
        opacity: 0;
        transform: translateX(50px);
    } to {
        opacity: 1;
        transform: translateX(0px);
    }
}
<div class="logo">
                    <h4>xxx</h4>    
            </div>
            <ul class="nav-links">
                <li class=".home-btn"><a href="#">xxx</a></li>
                <li class=".exchange-btn"><a href="#">xxx</a></li>
                <li class=".debit-btn"><a href="#">xxx</a></li>
                <li class=".crypto-btn"><a href="#">xxx</a></li>
                <li class=".stock-btn"><a href="#">xxx</a></li>
            </ul>
            <div class="burger">    
                <div class="line1"></div>
                <div class="line2"></div>
                <div class="line3"></div>

            </div>

Ваша помощь очень ценится :)

1 Ответ

1 голос
/ 20 июня 2020

Ваш цвет фона для этого класса был таким же, а также ваш javascript для этой функции имел некоторые проблемы. Вот как должно быть: 

const selectedNav = document.querySelectorAll('li');

const navChange = (index) => {
    selectedNav.forEach((item)=>{
        item.classList.remove('add-this-to-selected-section');
    })
    selectedNav[index].classList.add('add-this-to-selected-section');
}


selectedNav.forEach((item, index)=>{
    item.addEventListener('click', ()=> navChange(index))
})



const navSlide = () =>{
    const burger = document.querySelector('.burger');
    const nav = document.querySelector('.nav-links');
    const navLinks = document.querySelectorAll('.nav-links li');

// Toggle Nav
    burger.addEventListener('click',()=>{
        nav.classList.toggle('nav-active');


    //Animate Links
    navLinks.forEach((link, index)=>{
        if(link.style.animation){
            link.style.animation = '';
        } else {
        link.style.animation = `navLinkFade 0.5s ease forwards ${index / 7 + 0.5}s`
        }
   
    })
    //Burger Animation

    burger.classList.toggle('toggle');

})

}

navSlide();
nav{
    display: flex;
    justify-content: space-around;
    align-items: center;
    min-height: 8vh;
    font-family: 'Poppins', sans-serif;
    background-color: #ffffff;
}


.nav-links{
    display: flex;
    justify-content: space-around;
    width: 50%;
}

.nav-links li{
   list-style: none;
}

.nav-links a{
    color: #245871;
    text-decoration: none;
    letter-spacing: 3px;
    font-weight: 800;
    font-size: 18px;
}

.add-this-to-selected-section{
    color: white;
    background-color: red;
    padding: 5px;
    border-radius: 5px;
}

.burger div{
    width: 25px;
    height: 3px;
    background-color: #245871;
    margin: 5px;
    transition: all 0.3s ease;
}

.burger{
    display: none;
    cursor: pointer;
}


@media screen and (max-width: 1024){
    .nav-links{
        width: 70%;
        cursor: pointer;
        font-weight: 1200;
    }
}

@media screen and (max-width: 997px){
   body{
       overflow-x: hidden;
   }
   
   nav{
       color: white;
   }

   .head-btn{
       color: white;
   }

    .nav-links{
        position: absolute;
        right: 0px;
        height: 92vh;
        top: 8vh;
        background-color: #245871;
        color: white;
        display: flex;
        flex-direction: column;
        align-items: center;
        width: 70%;
        transform: translateX(100%);
        transition: transform 0.5s ease-in;
    }

    .nav-links a{
        opacity: 1;
        color: white;
    }
    .burger{
      display: block;
    }
}

.nav-active{
    transform: translateX(0%);
    color: white;
}

@keyframes navLinkFade{
    from{
        opacity: 0;
        transform: translateX(50px);
    } to {
        opacity: 1;
        transform: translateX(0px);
    }
}
<div class="logo">
                    <h4>xxx</h4>    
            </div>
            <ul class="nav-links">
                <li class=".home-btn"><a href="#">xxx</a></li>
                <li class=".exchange-btn"><a href="#">xxx</a></li>
                <li class=".debit-btn"><a href="#">xxx</a></li>
                <li class=".crypto-btn"><a href="#">xxx</a></li>
                <li class=".stock-btn"><a href="#">xxx</a></li>
            </ul>
            <div class="burger">    
                <div class="line1"></div>
                <div class="line2"></div>
                <div class="line3"></div>

            </div>
...