香草javascript轮播无法滑动

时间:2019-07-27 14:34:51

标签: javascript html css carousel

我有一个带主要div的简单轮播。在主div内有一个宽度为6000像素的ul,每当我单击向右或向左箭头时,该ul就会滑动。

我在两个箭头上都附加了事件侦听器,因此当我单击向左箭头时,我会将300px添加到该ul(slider)的当前marginLeft中,向右箭头添加marginRight 300px。

问题1:当我单击向左箭头时ul(滑块)移动,但是当我第二次单击时它仅移动1次,即使我向marginLeft添加300px而不是仅添加300px,它也没有移动,这是我常见的错误

问题2:当我单击它时,向右箭头根本无法滑动ul(滑块)。

Codepen:https://codepen.io/anon/pen/bXBaYW?editors=0110

const carousel = document.getElementById('carousel');
const leftArrow = carousel.querySelector('.carousel-left-arrow');
const rightArrow = carousel.querySelector('.carousel-right-arrow');
const slides = carousel.querySelector('.slides');

function scrollLeft() {
    slides.style.marginLeft += '300px';
    console.log(slides.style.marginLeft);
}

function scrollRight() {
    slides.style.marginLeft -= '300px';
    console.log(slides.style.marginLeft);
}

leftArrow.addEventListener('click' , scrollLeft);
rightArrow.addEventListener('click' , scrollRight);

1 个答案:

答案 0 :(得分:1)

首先,您应该像这样增加:

 let marginLeft = 0; /* first define this as global */
 marginLeft += 300;
 slides.style.marginLeft = marginLeft + "px";

由于"300px" + "300px" = "300px300px"的使用方式,这是错误的。

您将无法获得element.style.marginLeft之类的样式。

应使用getComputedStyle(element).property

这可行。

const carousel = document.getElementById('carousel');
const leftArrow = carousel.querySelector('.carousel-left-arrow');
const rightArrow = carousel.querySelector('.carousel-right-arrow');
const slides = carousel.querySelector('.slides');

let marginLeft = 0;

function scrollLeft() {
  marginLeft += 300;
  slides.style.marginLeft = marginLeft + "px";
  console.log(getComputedStyle(slides).marginLeft);
}

function scrollRight() {
  marginLeft -= 300;
  slides.style.marginLeft = marginLeft + "px";
  console.log(getComputedStyle(slides).marginLeft);
}

leftArrow.addEventListener('click', scrollLeft);
rightArrow.addEventListener('click', scrollRight);
#carousel {
  width: 90%;
  margin: 0 auto;
  margin-top: 20px;
  overflow: hidden;
  position: relative;
}

#carousel .arrow {
  position: absolute;
  top: 32%;
  background-color: rgba(255, 255, 255, 0.7);
  border-radius: 0%;
  cursor: pointer;
  width: 20px;
}

#carousel .arrow img {
  margin-top: 10px;
  max-width: 100%;
}

#carousel .carousel-left-arrow {
  width: 25px;
  left: 0;
  margin-left: 5px;
}

#carousel .carousel-right-arrow {
  right: 0;
  width: 25px;
}

#carousel .slides {
  width: 6000px;
  overflow: hidden;
  list-style: none;
  padding: 0;
  transition: 0.2s;
  margin-left: 0;
  margin-right: 0
}

#carousel .slide {
  float: left;
  margin: 0 5px;
  text-decoration: none;
  transform: 0.2s;
  color: whitesmoke;
}

#carousel .slides img {
  width: 300px
}
<div id="carousel">
  <span class="carousel-left-arrow arrow"><</span>
  <ul class="slides">
    <a href='#' class="slide">
      <img src="https://images.pexels.com/photos/248797/pexels-photo-248797.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500">
    </a>
    <a href='#' class="slide">
      <img src="https://images.pexels.com/photos/248797/pexels-photo-248797.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500">
    </a>
    <a href='#' class="slide">
      <img src="https://images.pexels.com/photos/248797/pexels-photo-248797.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500">
    </a>
    <a href='#' class="slide">
      <img src="https://images.pexels.com/photos/248797/pexels-photo-248797.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500">
    </a>
    <a href='#' class="slide">
      <img src="https://images.pexels.com/photos/248797/pexels-photo-248797.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500">
    </a>
    <a href='#' class="slide">
      <img src="https://images.pexels.com/photos/248797/pexels-photo-248797.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500">
    </a>
  </ul>
  <span class="carousel-right-arrow arrow">></span>
</div>