导航栏更改颜色javascript不工作

Question

在这里，我尝试在向下滚动时修改导航栏。我读到这个问题：

滚动后更改导航栏颜色？

向下滚动时导航栏中的转换

引导导航栏将颜色更改为滚动

我不能让它在我的网站上工作，我不明白这个问题：

HTML:

<!-- Navbar -->
<nav class="navbar justify-content-center navbar-expand-sm navbar-dark fixed-top navbar-custom">
    <!-- Menu Links -->
    <ul class="navbar-nav" >
        <li class="nav-item">
          <a class="nav-link" href="#">Services</a>
        </li>
        <li class="nav-item">
          <a class="nav-link" href="#">Portfolio</a>
        </li>
            <!-- Wanted Logo -->
        <a class="navbar-brand" href="#">
        <img src="img/logo-light.png" alt="wanted_logo" style="width: 3vw;">
        </a>
        <li class="nav-item">
          <a class="nav-link" href="#">About us</a>
        </li>
        <li class="nav-item">
          <a class="nav-link" href="#">Contact</a>
        </li>
    </ul>
</nav>
<!-- End Navbar -->

CSS：

.navbar-custom {
    background-color: rgba(0,0,0,0.5);
}

.navbar-custom ul li{
    text-transform: uppercase;
    font-size: 14px;
    color: #fff;
    padding-top: 34px;

}

.navbar-custom img{
    margin-left: 20px;
    margin-right: 7px;
}

.navbar-custom.scrolled {
    background-color: red !important;
    transition: background-color 200ms linear;
}

JS：

<script>$(function () {
  $(document).scroll(function () {
    var $nav = $(".navbar-custom");
    $nav.toggleClass("scrolled", $(this).scrollTop() > $nav.height());
  });
});
</script>

根据我的理解，当向下滚动超过导航栏的大小时，它应该会改变颜色。

我认为这可能是一个问题，因为导航栏的高度在这里没有明确定义，所以我尝试添加：

CSS

.navbar-custom{
    background-color: rgba(0,0,0,0.5);
    height: 100px;
}

它仍然不起作用，所以我也尝试使用这里提出的JS的另一个版本：滚动后更改导航条颜色？

JS:

<script type="text/javascript">
    $(document).ready(function(){
      $(window).scroll(function() {
        if ($(document).scrollTop() > 100) {
          $(".navbar-custom").css("background-color", "#f8f8f8");
        } else {
          $(".navbar-custom").css("background-color", "blue");
        }
      });
    });
    </script>

同样的结果没有任何改变。

我是新使用JS的，这里什么不起作用？

Answer 1

最后，我使用以下方法实现了它（作为w3返回顶部按钮教程的一部分）：

JS:

<script>
    window.onscroll = function() {scrollFunction()};

    function scrollFunction() {
        var $nav = $(".navbar-custom");

    if (document.body.scrollTop > 20 || document.documentElement.scrollTop > 50) {
        $nav.toggleClass("scrolled", $(this).scrollTop() > $nav.height());
    }
    }
</script>

链接到w3school返回顶部按钮：https://www.w3schools.com/howto/howto_js_scroll_to_top.asp

非常感谢。

Answer 2

这是因为你使用了不一致的滚动。你有的

$(window).scroll(function() {...});

不管你怎么检查它

if ($(document).scrollTop() > 100) {...}

你还有：

$(document).ready(function() {...});

将这些都设置为$（窗口）或$（文档），它就会工作。