RusRogo / RusRogo.github.io

Portfolio
0 stars 0 forks source link

Макет #3

Open ZvezdocheTag opened 6 years ago

ZvezdocheTag commented 6 years ago

Верстка слайдера:

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

На логотип hover эффект underline не нужно, логотип зачастую кликабельный но без эффектов.

ZvezdocheTag commented 6 years ago

Секция class=news , там ты используешь margin-bottom это ок, а потом margin-top задаешь в дочернем элементе class=news__item, непонятно

RusRogo commented 6 years ago

overlay вынес потому, что он в макете на всю ширину, как и фон для каждого блока (wrapper в адаптиве) Как-то можно по-другому? home_768px

RusRogo commented 6 years ago

Логотип исправил class=news исправил Я еще одну вещь понял - адаптив верстать нужно - каждый блок во всех разрешениях сразу. А не сверстать (как я) макет полностью на 320px, потом на 480px - всю конструкцию блоков переделывать Много времени убил

ZvezdocheTag commented 5 years ago

overlay вынес потому, что он в макете на всю ширину, как и фон для каждого блока (wrapper в адаптиве) Как-то можно по-другому? home_768px

Да, overlay принадлежит блоку wrapSlider, соответственно он должен там распологаться. Это можно реализовать с помощью псевдоэлементов :before, :after

ZvezdocheTag commented 5 years ago

Логотип исправил class=news исправил Я еще одну вещь понял - адаптив верстать нужно - каждый блок во всех разрешениях сразу. А не сверстать (как я) макет полностью на 320px, потом на 480px - всю конструкцию блоков переделывать Много времени убил

Тут с опытом приходит понимание как тебе удобнее и лучше. Вообще практика mobile first удобная, но к ней привыкнуть нужно и чуть больше практики