Контейнер для слайд-шоу одинаковой высоты

У меня есть слайд-шоу с четырьмя изображениями разной высоты. Я пытаюсь сохранить у них одинаковую высоту 700 пикселей, но не могу понять, как это сделать. Я установил img на max-height: 700px, но, похоже, это не сработало.

Вот кодовый ключ.

   

 var slideIndex = 1;

  showSlide(slideIndex);


  function plusSlides(n) {
      showSlide(slideIndex += n);
  }

  function currentSlide(n) {
      showSlide(slideIndex = n);
  }


  function showSlide(n) {

      var i;

      var slides = document.getElementsByClassName("myslides");

      var dots = document.getElementsByClassName("dots");

      if (n > slides.length) {
          slideIndex = 1
      };

      if (n < 1) {
          slideIndex = slides.length
      };

      for (i = 0; i < slides.length; i++) {

          slides[i].style.display = "none";

      };

      for (i = 0; i < dots.length; i++) {

          dots[i].className = dots[i].className.replace(" active", "");

      };

      slides[slideIndex - 1].style.display = "block";

      dots[slideIndex - 1].className += " active";

  }
body{
  font-family: verdana,sans-serif;
  margin: 0;
  font-size: 100%;
}

#slideshow-container{
  position: relative;
  margin: auto;
  background-color: black;
  overflow: hidden;
}

#slideshow-container img{
  display: block;
  margin: 0 auto;
  max-width: 100%;
  position: relative;
  max-height: 700px !important;
}

.myslides{
  display: none;
}

.prev , .next{
  position: absolute;
  top: 50%;
  font-size: 30px;
  font-weight: bold;
  padding: 16px;
  margin-top: -22px;
  border-radius: 0 3px 3px 0;
  color: #fff;
  cursor: pointer;
  z-index: 1;
}

.next{
  right: 0;
  border-radius: 3px 0 0 3px;
}

.prev:hover,.next:hover{
  background-color: rgba(0,0,0,0.8);
}

.caption{
  text-align: center;
  position: absolute;
  bottom: 5px;
  width: 100%;
  color: #f2f2f2;
  font-size: 1em;
/*   font-size: 15px; */
/*   padding: 8px 22px; */
}

.fade{
  animation-name:fade;
  animation-duration:1.5s;
}

@keyframes fade{
  from {opacity: 0.4;}
  to {opacity: 1;}
}
<script src = "https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<!---Images not owned by me --->

<div id = "slideshow-container">
 <div id = "mainImg">

  <div class = "myslides fade">
    <div><img src = "https://static.boredpanda.com/blog/wp-content/uploads/2014/12/Top-10-photographers-for-travel-portraits27__700.jpg"></div>
      <div class = "caption">Text 1</div>
  </div>

  <div class = "myslides fade">
    <div><img src = "https://cdn-images-1.medium.com/max/1600/1*way-yv9effv2b7PKZeUDMA.png"></div>
      <div class = "caption">Text 2</div>
  </div>

  <div class = "myslides fade">
    <div><img src = "https://www.shutterbug.com/images/17/promosa111217.png"></div>
      <div class = "caption">Text 3</div>
  </div>

  <div class = "myslides fade">
    <div><img src = "http://jmacpratt.weebly.com/uploads/1/2/0/1/12013129/portrait2_orig.jpg"></div>
      <div class = "caption">Text 4</div>
  </div>

    <a class = "prev" onclick = "plusSlides(-1)">&#10094;</a>

    <a class = "next" onclick = "plusSlides(1)">&#10095;</a>

   </div>
</div>

Здравствуйте, либо попробуйте удалить изображения и сделать их как background-img с обложкой, либо вырезать изображения, чтобы они были одинаковой ширины и высоты

Sara Kat 13.06.2018 15:10

замените max-height на height: 700px; это работает.

ruchika jain 13.06.2018 15:24
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
0
2
721
3

Ответы 3

Измените стиль max-height: 700px на height: 700px и width: 100%; на width: auto;

var slideIndex = 1;

showSlide(slideIndex);


function plusSlides(n){

showSlide(slideIndex += n);

}


function currentSlide(n) {

showSlide(slideIndex = n);

}


function showSlide(n){

var i;

var slides = document.getElementsByClassName("myslides");

var dots = document.getElementsByClassName("dots");

if (n > slides.length) { slideIndex = 1};

if (n < 1) { slideIndex = slides.length};

for (i=0;i<slides.length;i++) {

slides[i].style.display = "none";

};

for (i=0;i<dots.length;i++) {

dots[i].className = dots[i].className.replace(" active","");

};

slides[slideIndex-1].style.display = "block";

dots[slideIndex-1].className += " active";

}
body{
  font-family: verdana,sans-serif;
  margin: 0;
  font-size: 100%;
}

#slideshow-container{
  position: relative;
  margin: auto;
  background-color: black;
  overflow: hidden;
}

#slideshow-container img{
  display: block;
  margin: 0 auto;
  width: auto;
  position: relative;
  height: 700px !important;
}

.myslides{
  display: none;
}

.prev , .next{
  position: absolute;
  top: 50%;
  font-size: 30px;
  font-weight: bold;
  padding: 16px;
  margin-top: -22px;
  border-radius: 0 3px 3px 0;
  color: #fff;
  cursor: pointer;
  z-index: 1;
}

.next{
  right: 0;
  border-radius: 3px 0 0 3px;
}

.prev:hover,.next:hover{
  background-color: rgba(0,0,0,0.8);
}

.caption{
  text-align: center;
  position: absolute;
  bottom: 5px;
  width: 100%;
  color: #f2f2f2;
  font-size: 1em;
/*   font-size: 15px; */
/*   padding: 8px 22px; */
}

.fade{
  animation-name:fade;
  animation-duration:1.5s;
}

@keyframes fade{
  from {opacity: 0.4;}
  to {opacity: 1;}
}
<script src = "https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<!---Images not owned by me --->

<div id = "slideshow-container">
 <div id = "mainImg">

  <div class = "myslides fade">
    <div><img src = "https://static.boredpanda.com/blog/wp-content/uploads/2014/12/Top-10-photographers-for-travel-portraits27__700.jpg"></div>
      <div class = "caption">Text 1</div>
  </div>

  <div class = "myslides fade">
    <div><img src = "https://cdn-images-1.medium.com/max/1600/1*way-yv9effv2b7PKZeUDMA.png"></div>
      <div class = "caption">Text 2</div>
  </div>

  <div class = "myslides fade">
    <div><img src = "https://www.shutterbug.com/images/17/promosa111217.png"></div>
      <div class = "caption">Text 3</div>
  </div>

  <div class = "myslides fade">
    <div><img src = "http://jmacpratt.weebly.com/uploads/1/2/0/1/12013129/portrait2_orig.jpg"></div>
      <div class = "caption">Text 4</div>
  </div>

    <a class = "prev" onclick = "plusSlides(-1)">&#10094;</a>

    <a class = "next" onclick = "plusSlides(1)">&#10095;</a>

   </div>
</div>

Отлично работает в полноэкранном режиме, но вам, вероятно, придется использовать media-queries для небольших устройств, поскольку изображение не достигает желаемой ширины кадра.

Alex 13.06.2018 15:14

Я бы предложил вместо этого использовать фоновое изображение, чтобы вы могли центрировать изображение, и было бы легче управлять для разных экранов.

John Baker 13.06.2018 15:24

@Nandita Arora Sharma Я внес предложенное вами изменение, но когда я изменяю размер окна, горизонтальные изображения обрезаются. Я также стараюсь, чтобы изображения были отзывчивыми, и, похоже, это отлично работает только для экрана рабочего стола.

user7311741 13.06.2018 15:24

Добавьте свойство height = "700px" во внешний контейнер. и то же свойство height = "700px" для каждого элемента img, а именно:

<script src = "https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<!---Images not owned by me --->

<div id = "slideshow-container" height = "700px">
 <div id = "mainImg">

  <div class = "myslides fade">
    <div><img height = "700px" src = "https://static.boredpanda.com/blog/wp-content/uploads/2014/12/Top-10-photographers-for-travel-portraits27__700.jpg"></div>
      <div class = "caption">Text 1</div>
  </div>

  <div class = "myslides fade">
    <div><img height = "700px" src = "https://cdn-images-1.medium.com/max/1600/1*way-yv9effv2b7PKZeUDMA.png"></div>
      <div  class = "caption">Text 2</div>
  </div>

  <div class = "myslides fade">
    <div><img height = "700px" src = "https://www.shutterbug.com/images/17/promosa111217.png"></div>
      <div class = "caption">Text 3</div>
  </div>

  <div class = "myslides fade">
    <div><img height = "700px" src = "http://jmacpratt.weebly.com/uploads/1/2/0/1/12013129/portrait2_orig.jpg"></div>
      <div class = "caption">Text 4</div>
  </div>

    <a class = "prev" onclick = "plusSlides(-1)">&#10094;</a>

    <a class = "next" onclick = "plusSlides(1)">&#10095;</a>

   </div>
</div>

Я бы предложил другой подход, добавив изображения в качестве фоновых, поскольку это упростило бы установку определенной высоты и сохранение ширины относительно нее.

var slideIndex = 1;

showSlide(slideIndex);


function plusSlides(n) {

  showSlide(slideIndex += n);

}


function currentSlide(n) {

  showSlide(slideIndex = n);

}


function showSlide(n) {

  var i;

  var slides = document.getElementsByClassName("myslides");

  var dots = document.getElementsByClassName("dots");

  if (n > slides.length) {
    slideIndex = 1
  };

  if (n < 1) {
    slideIndex = slides.length
  };

  for (i = 0; i < slides.length; i++) {

    slides[i].style.display = "none";

  };

  for (i = 0; i < dots.length; i++) {

    dots[i].className = dots[i].className.replace(" active", "");

  };

  slides[slideIndex - 1].style.display = "block";

  dots[slideIndex - 1].className += " active";

}
body {
  font-family: verdana, sans-serif;
  margin: 0;
  font-size: 100%;
}

#slideshow-container {
  position: relative;
  margin: auto;
  background-color: black;
  overflow: hidden;
}

#slideshow-container>div {
  position: relative;
  height: 700px;
  width: 100%;
}

#slideshow-container .bg-img {
  background-repeat: no-repeat;
  background-size: auto 100%;
  background-position: center;
  position: absolute;
  width: 100%;
  height: 100%;
  top: 0;
  left: 0;
}

#slideshow-container img {
  display: block;
  margin: 0 auto;
  max-width: 100%;
  position: relative;
  max-height: 700px !important;
}

.myslides {
  display: none;
}

.prev,
.next {
  position: absolute;
  top: 50%;
  font-size: 30px;
  font-weight: bold;
  padding: 16px;
  margin-top: -22px;
  border-radius: 0 3px 3px 0;
  color: #fff;
  cursor: pointer;
  z-index: 1;
}

.next {
  right: 0;
  border-radius: 3px 0 0 3px;
}

.prev:hover,
.next:hover {
  background-color: rgba(0, 0, 0, 0.8);
}

.caption {
  text-align: center;
  position: absolute;
  bottom: 5px;
  width: 100%;
  color: #f2f2f2;
  font-size: 1em;
  /*   font-size: 15px; */
  /*   padding: 8px 22px; */
}

.fade {
  animation-name: fade;
  animation-duration: 1.5s;
}

@keyframes fade {
  from {
    opacity: 0.4;
  }
  to {
    opacity: 1;
  }
}
<!---Images not owned by me --->

<div id = "slideshow-container">
  <div id = "mainImg">

    <div class = "myslides fade">
      <div>
        <span class = "bg-img" style = "background-image: url('https://static.boredpanda.com/blog/wp-content/uploads/2014/12/Top-10-photographers-for-travel-portraits27__700.jpg')"></span>
      </div>
      <div class = "caption">Text 1</div>
    </div>

    <div class = "myslides fade">
      <div>
        <span class = "bg-img" style = "background-image: url('https://cdn-images-1.medium.com/max/1600/1*way-yv9effv2b7PKZeUDMA.png')"></span>
      </div>
      <div class = "caption">Text 2</div>
    </div>

    <div class = "myslides fade">
      <div>
        <span class = "bg-img" style = "background-image: url('https://www.shutterbug.com/images/17/promosa111217.png')"></span>
      </div>
      <div class = "caption">Text 3</div>
    </div>

    <div class = "myslides fade">
      <div>
        <span class = "bg-img" style = "background-image: url('http://jmacpratt.weebly.com/uploads/1/2/0/1/12013129/portrait2_orig.jpg')"></span>
      </div>
      <div class = "caption">Text 4</div>
    </div>

    <a class = "prev" onclick = "plusSlides(-1)">&#10094;</a>

    <a class = "next" onclick = "plusSlides(1)">&#10095;</a>

  </div>
</div>

<br>

 <div style = "text-align:center">

<span class = "dots" onclick = "currentSlide(1)"></span> 

<span class = "dots" onclick = "currentSlide(2)"></span> 

<span class = "dots" onclick = "currentSlide(3)"></span> 

<span class = "dots" onclick = "currentSlide(4)"></span> 

</div>

Другие вопросы по теме