Jump to content

Оцените, пожалуйста верстку.


codeliner
 Share

Recommended Posts

На выходных сверстал, если можно так выразиться, еще одну страницу.

Макет в png  http://codeliner.ucoz.com/fadelicious/home.png

Макет в psd  https://yadi.sk/i/Td7IVc0pqKKNc

Сверстанная веб-страница  http://codeliner.ucoz.com/fadelicious/index.html

Прошу объективной критики и дельных советов на будущее :)

Edited by codeliner
Link to comment
Share on other sites

Хм, более 30 просмотров и ни одного ответа. В других темах, я смотрю, коээфициент - от 10 до 20 просмотров на ответ. Интересно, это связано с простотой страницы, или еще с чем-нибудь <_<

Link to comment
Share on other sites

1. Логотип должен быть ссылкой и в теге img, он не должен добавляться через css.
2. Блокам offer я бы дал обертку div.wrap-offer и ему задал margin-top: 50px; а не каждому блоку давать отступ сверху и так как там вы используете float: left; то нужно убрать обтекание у блока.
3. Много ненужных свойств для блока global-container http://pixs.ru/showimage/2016032222_3973083_21223897.png
4. Ненужно давать чуть ли ни каждому блоку border: 0px solid black;
5. Изображения в слайдере должны добавляться через тег img.
6. Также не понятно почему вы дали копирайту left: -50%; margin-left: 170px; если он без этих свойств выглядит так как по макету.
7. Шрифт в меню, кнопках 
View details и в тексте приветствия не такой как в макете.
8. Зачем давать блоку second-footer width: 200%;  left: -50%; ?
9. Зачем в head тег <style> ?

Edited by Spotlight
Link to comment
Share on other sites

1. Я бы в блоках offer кнопку "view details" делал обычной ссылкой, а не тегом button - зачем он там?

2. Зачем вешать столько свойств на абзац? http://prntscr.com/ainp4t 

И что вообще дает width и height:auto?

Link to comment
Share on other sites

В 22.03.2016 в 00:10, Spotlight сказал:

1. Логотип должен быть ссылкой и в теге img, он не должен добавляться через css.

Полностью согласен :)

В 22.03.2016 в 00:10, Spotlight сказал:

2. Блокам offer я бы дал обертку div.wrap-offer и ему задал margin-top: 50px; а не каждому блоку давать отступ сверху и так как там вы используете float: left; то нужно убрать обтекание у блока.

Тоже об этом подумал, когда уже сделал страницу. У меня из-за того, что они идут без обертки произошло схлопывание, из-за чего были проблемы с размещением футера. Поэтому у global-containerа идет overflow:auto.

В 22.03.2016 в 00:10, Spotlight сказал:

3. Много ненужных свойств для блока global-container http://pixs.ru/showimage/2016032222_3973083_21223897.png

Да, но height:auto нужно, т.к. без него футер правильно не позиционируется. Overflow:auto тоже нужно, о чем я писал выше.

 

В 22.03.2016 в 00:10, Spotlight сказал:

4. Ненужно давать чуть ли ни каждому блоку border: 0px solid black;

Просто изначально я даю каждому блоку border:1px solid black, чтобы лучше видеть его очертания и, таким образом, нагляднее видеть результат кода. А потом я это свойство не удаляю, а лишь обнуляю, чтобы в случае необходимости можно было бы быстрее включить границу

 

В 22.03.2016 в 00:10, Spotlight сказал:

5. Изображения в слайдере должны добавляться через тег img.

Почему обязательно так? Или вы имеете в виду, что эти изображения должны быть ссылками?

 

В 22.03.2016 в 00:10, Spotlight сказал:

6. Также не понятно почему вы дали копирайту left: -50%; margin-left: 170px; если он без этих свойств выглядит так как по макету.

Там было разное отображение в различных браузерах, поэтому пришлось ставить такие заплатки. Поэтому, кстати, подключены еще отдельные стили для IE, для Firefox и для Safari. По другому как-то не получалось.

В 22.03.2016 в 00:10, Spotlight сказал:

7. Шрифт в меню, кнопках View details и в тексте приветствия не такой как в макете.

Это да, шрифты я подключать еще пока не научился.

 

В 22.03.2016 в 00:10, Spotlight сказал:

8. Зачем давать блоку second-footer width: 200%;  left: -50%; ?

Чтобы скрыть внутреннюю тень слева и справа (т.к. она внутренняя, то способ с псевдоэлементом не прокатывает). По этой же причине его высота на 20 пикселей больше, чем видимая, и он смещен на 20 пикселей вверх (margin-top:-20px).

 

В 22.03.2016 в 00:10, Spotlight сказал:

9. Зачем в head тег <style>

Видимо забыл удалить (подключал внутренние стили, когда не срабатывали подключенные - там проблемы со специфичностью были).

 

В 22.03.2016 в 00:36, alejandro13 сказал:

Я бы в блоках offer кнопку "view details" делал обычной ссылкой, а не тегом button - зачем он там?

Это да, согласен.

В 22.03.2016 в 00:36, alejandro13 сказал:

2. Зачем вешать столько свойств на абзац? http://prntscr.com/ainp4t 

И что вообще дает width и height:auto?

Ну там процентов 80 свойств нужные. А width, и height auto - назначаю просто, чтобы проговаривать про себя, что у блока вот эти свойства, и к чему это приводит и.т.д. - в учебных целях, одним словом.

 

В 24.03.2016 в 15:52, Int сказал:

Видимо, это чтобы не делать transform: translateX(-50%)

Похоже, можно было и так сделать, но ширину все равно больше 100% надо назначать - и так и так.

 

В 24.03.2016 в 15:52, Int сказал:

У меню Contact справа полоска осталась.

Забыл указать li:last-child a {background:none;}

Спасибо всем за ответы!

Edited by codeliner
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

  • Similar Content

    • By Салават
      Есть слайдер на сайте
      <amp-carousel width="450" height="300" layout="responsive" type="slides" loop="" id="wizard-carouselsbxJ62ccHm" on="slideChange:AMP.setState({selectedsbxJ62ccHm: {slide: event.index}})" data-autoplay="true" data-delay="3000" autoplay="" delay="3000"> <div class="item slider-image mbr-flex"> <div class="item-wrapper"> <amp-img src="assets/images/destination-min-506x337.jpeg" layout="responsive" width="450.4451038575668" height="300" alt="image" class="placeholder-loader lb-image"> <span class="mbr-text mbr-fonts-style display-7"> Предназначение: где мы находимся и куда наступать</span> <div placeholder="" class="placeholder"> <div class="mobirise-spinner"> <em>Предназначение</em> <em>Где мы находимся и куда наступать</em> <em>Причина, по которой я просыпаюсь</em> </div></div> </amp-img> </div> </div><div class="item slider-image mbr-flex"> <div class="item-wrapper"> <amp-img src="assets/images/-min-640x426.jpg" layout="responsive" width="450.70422535211264" height="300" alt="image" class="placeholder-loader lb-image"> <span class="mbr-text mbr-fonts-style display-7"> Управление временем и задачами: насколько вы эффективны?</span> <div placeholder="" class="placeholder"> <div class="mobirise-spinner"> <em>Управление временем и задачами</em> <em>Насколько вы эффективны?</em> <em></em> </div></div> </amp-img> </div> </div><div class="item slider-image mbr-flex"> <div class="item-wrapper"> <amp-img src="assets/images/mbr-1920x1371.jpg" layout="responsive" width="420.13129102844636" height="300" alt="image" class="placeholder-loader lb-image"> <span class="mbr-text mbr-fonts-style display-7"> Целеполагание: что происходит и как на это повлиять?</span> <div placeholder="" class="placeholder"> <div class="mobirise-spinner"> <em>Целеполагание</em> <em>Что происходит?</em> <em>что происходит</em> </div>Как на это повлиять?</div> </amp-img> </div> </div><div class="item slider-image mbr-flex"> <div class="item-wrapper"> <amp-img src="assets/images/-min-640x426.jpg" layout="responsive" width="450.70422535211264" height="300" alt="image" class="placeholder-loader lb-image"> <span class="mbr-text mbr-fonts-style display-7"> Интернетмаркетинг: страшное слово?</span> <div placeholder="" class="placeholder"> <div class="mobirise-spinner"> <em>Интернетмаркетинг</em> <em>Интернетмаркетинг - это искусство изъятия денег из населения в интернете</em> <em></em> </div></div> </amp-img> </div> </div><div class="item slider-image mbr-flex"> <div class="item-wrapper"> <amp-img src="assets/images/-min-640x348.jpg" layout="responsive" width="551.7241379310344" height="300" alt="image" class="placeholder-loader lb-image"> <span class="mbr-text mbr-fonts-style display-7"> Продажи и переговоры</span> <div placeholder="" class="placeholder"> <div class="mobirise-spinner"> <em>Продажи</em> <em>Переговоры</em> <em></em> </div></div> </amp-img> </div> </div><div class="item slider-image mbr-flex"> <div class="item-wrapper"> <amp-img src="assets/images/web-master-min-640x426.jpg" layout="responsive" width="450.70422535211264" height="300" alt="image" class="placeholder-loader lb-image"> <span class="mbr-text mbr-fonts-style display-7"> Вебмастерство: секреты из тайника</span> <div placeholder="" class="placeholder"> <div class="mobirise-spinner"> <em>Вебмастер</em> <em>Копирайтер</em> <em>Вебдизайнер</em> </div></div> </amp-img> </div> </div> </amp-carousel> это слайдер. Надо в этом слайдере сделать картинки кликабельными. Кто подскажет куда и какой код вставить, чтобы слайдер стал кликабельным. Можно даже в ЛС. Спасибо.
    • By Darya
      Ребята, всем доброго времени суток) Помогите пожалуйста со слайдером! В  javascript плохо разбираюсь((
      Нашла подходящее мне решение, но никак не могу настроить под себя.
      Нашла здесь: http://lantana-studio.ru/howto/simpleslider/
      У меня содержимое слайдера вылезло из контейнера, фото прикреплю
      Вот код HTML
      <div class="slider"> <div class="slider__block d-flex__row"> <div class=""> <img src="img/otziv-img.png" alt="Фото" class="slider__img"> </div> <div class="slider__block_text"> <div class="slider__block_text-row d-flex__widthHeight-center"> <p class="slider__block_fio"> 1Диана Апасова </p> <p class="slider__block_vk"> vk.com/apasova </p> <p class="slider__block_place"> Москва → София </p> </div> <div class="slider__block_text-description"> Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in </div> </div> </div> <div class="slider__block d-flex__row"> <div class=""> <img src="img/otziv-img.png" alt="Фото" class="slider__img"> </div> <div class="slider__block_text"> <div class="slider__block_text-row d-flex__widthHeight-center"> <p class="slider__block_fio"> 2Диана Апасова </p> <p class="slider__block_vk"> vk.com/apasova </p> <p class="slider__block_place"> Москва → София </p> </div> <div class="slider__block_text-description"> Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in </div> </div> </div> <div class="slider__block d-flex__row"> <div class=""> <img src="img/otziv-img.png" alt="Фото" class="slider__img"> </div> <div class="slider__block_text"> <div class="slider__block_text-row d-flex__widthHeight-center"> <p class="slider__block_fio"> 3Диана Апасова </p> <p class="slider__block_vk"> vk.com/apasova </p> <p class="slider__block_place"> Москва → София </p> </div> <div class="slider__block_text-description"> Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in </div> </div> </div> <a class="prev" onclick="minusSlide()">◀</a> <a class="next" onclick="plusSlide()">▶</a> </div> <div class="slider-dots"> <span class="slider-dots_item" onclick="currentSlide(1)"></span> <span class="slider-dots_item" onclick="currentSlide(2)"></span> <span class="slider-dots_item" onclick="currentSlide(3)"></span> </div> CSS
      .slider { position: relative; } .slider__block { width: 840px; height: 200px; margin: 0px auto; padding: 10px 20px; box-shadow: 0 5px 8px -5px #000; } .slider__img { margin: 20px 20px 0px 0px; } .slider__block_text { margin-top: 15px; } .slider__block_fio { text-transform: uppercase; font-size: 1.067rem; font-family: 'MuseoSansCyrl-900'; margin-right: 33px; margin-left: -64px; } .slider__block_vk { text-transform: uppercase; font-size: 0.8rem; font-family: 'MuseoSansCyrl-900'; margin-right: 140px; color: #23aae2; border-bottom: 2px solid #cce6f6; } .slider__block_place { text-transform: uppercase; font-size: 0.8rem; font-family: 'MuseoSansCyrl-100'; color: #3eb0e4; } .slider__block_text-description { font-size: 0.933rem; font-family: 'MuseoSansCyrl-100Italic'; color: #363636; } /* Кнопки вперед и назад */ .slider .prev, .slider .next { cursor: pointer; position: absolute; top: 0; top: 50%; width: auto; margin-top: -22px; padding: 16px; color: #000; font-weight: bold; font-size: 18px; transition: 0.6s ease; border-radius: 0 3px 3px 0; } .slider .next { right: 0; border-radius: 3px 0 0 3px; } /* При наведении на кнопки добавляем фон кнопок */ .slider .prev:hover, .slider .next:hover { background-color: rgba(0, 0, 0, 0.8); } /* Кружочки */ .slider-dots { text-align: center; margin-top: 20px; } .slider-dots_item{ cursor: pointer; height: 12px; width: 12px; margin: 0 2px; background-color: #ddd; border-radius: 50%; display: inline-block; transition: background-color 0.6s ease; } .active, .slider-dots_item:hover { background-color: #aaa; } /* Анимация слайдов */ .slider .slider__block { -webkit-animation-name: fade; -webkit-animation-duration: 1.5s; animation-name: fade; animation-duration: 1.5s; } @-webkit-keyframes fade { from { opacity: .4 } to { opacity: 1 } } @keyframes fade { from { opacity: .4 } to { opacity: 1 } } JS
      /* Индекс слайда по умолчанию */ var slideIndex = 1; showSlides(slideIndex); /* Функция увеличивает индекс на 1, показывает следующй слайд*/ function plusSlide() { showSlides(slideIndex += 1); } /* Функция уменьшяет индекс на 1, показывает предыдущий слайд*/ function minusSlide() { showSlides(slideIndex -= 1); } /* Устанавливает текущий слайд */ function currentSlide(n) { showSlides(slideIndex = n); } /* Основная функция слайдера */ function showSlides(n) { var i; var slides = document.getElementsByClassName("slider__block"); var dots = document.getElementsByClassName("slider-dots_item"); 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"; } Прошу не судить строго, я только в начале своего пути))
      Буду очень благодарна, за помощь!!

    • By Takono
      Здравствуйте. Создан слайдер из трёх слайдов. На слое над слайдером с помощью :before создан контент (один блок, содержащий текст, меню со ссылками и форму). Однако при адаптации под разные разрешения всё очень сильно съезжает и искажается. Есть ли какой-то более рациональный способ разместить слой с контентом над слайдером, чтобы слайды переключались под недвижимым контентом? Подскажите пожалуйста.
      id="main" это блок, который находится в слое над id="slider"
       
      index.html
      style.css
    • By Bubunt
      Здравствуйте, вопрос к уважаемым специалистам. Только приступил к изучению html+css. Знаю что можно реализовать такой слайдер на чистом html+css, но мне интересно, как сделать такой слайдер с помощью OWL Carousel 2? В частности, как правильно для данной секции написать html код и как правильно прописать стили для слайдера?  
      По одному ролику на YouTube получилось сделать так, дальше моих знаний уже не хватает. Отмечу, что делаю разметку страницы, до написания стилей еще не доходил. Вот мой код данной разметки, а тут макет.
      Буду благодарен за помощь, так как застрял именно со слайдером, во всей остальной разметке вроде разобрался.
    • By Aliona108
      Проблема со слайдером. При заходе на стартовою страницу  слайдер ведет себя не корректно. Посмотрите пожалуйста в чем проблема.. Скину архив всего сайта, чтобы была понятна проблема целиком. 
      Проект первый Низовцева Алена.7z
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy