我试图让我的导航栏在我向下滚动时向上缩回，然后在我向上滚动时再次出现，但它没有做任何事情

Question

我对编写代码非常陌生，但我看到它以同样的方式完成，我不明白为什么它什么都不做。 问题可能在 CSS 部分的某个地方。

JavaScript

var navbar = document.getElementsByClassName("navbar");
       var lastY = window.pageYOffset;
       window.onscroll = function()
      {
       var currY = window.pageYOffset;
       if (lastY > currY)
       {
        document.getElementsByClassName("navbar").style.top = "0";
       }
       else
       {
        document.getElementsByClassName("navbar").style.top = "-50px";
       }
       lastY = currY;
      }

HTML-我不完全确定为什么我必须将所有链接放在它们单独的导航栏项目类 div 中，但是如果我不这样做，它们就会开始重叠标题（我希望导航栏在左侧有该名称以及右侧的链接），并且还会使导航栏后页面的其他内容消失。

<div class="navbar">
        <h1>Квартална кръчма Тримата глупаци</h1>
        <div class="navbar-items">
          <div class="navbar-items"><a href="index.html">Home</a></div>
          <div class="navbar-items"><a style="color:red;" href="Menu.html">Menu</a></div>
          <div class="navbar-items"><a href="About_Us.html">About us</a></div>
          <div class="navbar-items"><a href="Contact_Us.html">Contact us</a></div>
        </div>
      </div>

CSS

.navbar {
    overflow: hidden;
    position: fixed;
    display: flex;
    align-items: center;
    justify-content: space-between;
    background:#505d61;
    z-index: 99;
    padding: 10px;
    height: 60px;
    top: 0;
    width: 100%;
    box-shadow: 0 0 10px black;
}

.navbar-items {
    overflow: hidden;
    float:left;
    display:block;
    margin-left: 40px;
    margin-right: 5px;
    gap: 80px;
    font-size: 21px;
    font-weight: 550;
}

.navbar a{
   color:black;
   text-decoration: none;
}

.navbar a:hover{ 
    color: white;
}

Answer 1

正如 CBroe 的评论所指出的那样，您的问题是您期望从document.getElementsByClassName("navbar")返回一个元素，但实际上您正在返回一个数组。 要访问您需要将其从数组中拉出的元素，您可以像document.getElementsByClassName("navbar")[0]一样执行此操作。

因此，将您的代码更新为

var lastY = window.pageYOffset;
window.onscroll = function(){
    var currY = window.pageYOffset;
    if (lastY > currY){
        document.getElementsByClassName("navbar")[0].style.top = "0";
    }
    else{
        document.getElementsByClassName("navbar")[0].style.top = "-50px";
    }
    lastY = currY;
}

应该解决问题。