Раскритикуйте вёрстку

[Oleg90s]

Здравствуйте, посмотрите пожалуйста качество вёрстки http://proba-verstki.xyz/ буду очень благодарен за конструктивную критику. Сайт не адаптивный, но я постарался сделать его кроссбраузерным.

P.S. Наверно раздел не тот, но в "обсуждении работ" надо 30 сообщений, которых у меня нет, поэтому решил запостить тут.

[green-vik]

привет, не плохо, но мне кажется верхнее меню (в шапке) и нижнее ссылки при наведении сделать выделяемые

[DivMan]

Сделай отступы, а то при сужении экрана, всё впритык 

 

Edited March 22, 2017 by DivMan

[Oleg90s]

8 часов назад, green-vik сказал:

привет, не плохо, но мне кажется верхнее меню (в шапке) и нижнее ссылки при наведении сделать выделяемые

Спасибо, мне тоже так кажется, но в макете этого нет

7 часов назад, DivMan сказал:

Сделай отступы, а то при сужении экрана, всё впритык 

Спасибо, сейчас поправлю

[radioactive]

Доктайп html5, используйте новые теги

<div class='header'> -> <header>
<div class='nav'> -> <nav>
<div class='footer'> -> <footer>

Книгам я бы сделал увеличение при наведении курсора мыши

[Oleg90s]

12 часа назад, radioactive сказал:

Доктайп html5, используйте новые теги

<div class='header'> -> <header>
<div class='nav'> -> <nav>
<div class='footer'> -> <footer>

Книгам я бы сделал увеличение при наведении курсора мыши

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

[Endorphin]

В форме поиска сделать бы отступ слева немного а то совсем в притык. И кстати в чем использование html5 тегов не кроссбраузерно, или вам нужны будут динозавры? Дак для этого есть html5shiv, и семантично и html5 и кроссбраузерно

[Oleg90s]

55 минут назад, Endorphin сказал:

В форме поиска сделать бы отступ слева немного а то совсем в притык. И кстати в чем использование html5 тегов не кроссбраузерно, или вам нужны будут динозавры? Дак для этого есть html5shiv, и семантично и html5 и кроссбраузерно

Спасибо, отступ сейчас сделаю. 

Эту вёрстку делал под динозавров, до ie6 включительно. Про html5shiv не знал, за это вам отдельное спасибо 

[Endorphin]

На здоровье!

[Hasiev]

Сетка ломается при масштабировании, и больше динамики нужно добавить .. Не важно указана она в макете или нет.

[Oleg90s]

2 часа назад, Hasiev сказал:

Сетка ломается при масштабировании, и больше динамики нужно добавить .. Не важно указана она в макете или нет.

Спасибо учту, но тогда вопрос: надо добиться чтоб сетка не ломалась прям до полного уменьшения/увеличения масштаба, доступного в браузере? У меня при 67% держится, я думал этого достаточно.

[Hasiev]

55 минут назад, Oleg90s сказал:

Спасибо учту, но тогда вопрос: надо добиться чтоб сетка не ломалась прям до полного уменьшения/увеличения масштаба, доступного в браузере? У меня при 67% держится, я думал этого достаточно.

Я сам не знаю зачем это, мне так гуру говорили .. Хотя в одном из ТЗ учебных макетов выложенных на этом форуме я прочитал следующее - 

Цитата

Масштабирование страниц
Поддержка не обязательна. Пользователь сам на себя берет отвественность за результат отображения страницы применяя к ней любые инструменты которые влияют на её изменение.