laura-313 / f1-3-c2p1-colmar-academy

0 stars 0 forks source link

use one media-container class on all images and the video #6

Open elisementor opened 7 years ago

elisementor commented 7 years ago

this gets reused a fair amount can instead make one class

.media-container {
  overflow: hidden;
}

.media-container img,
.media-container video {
  display: block;
  width: 100%;
}

then add width and padding specifics to instances of that class

.content-image-box .media-container {
  width: 35%;
}

@media only screen and (max-width: 895px) {

  .content-image-box .media-container {
    width: 100%;
  }
}

https://github.com/laura-313/f1-3-c2p1-colmar-academy/blob/master/colmar-academy-final/resources/css/style.css#L94-L100

laura-313 commented 7 years ago

your comments are thoughtful and helpful. thank you