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

[Killerblade]

Всем привет.

 

Верстаю в принипе давно, но вот сейчас хотел посоветоваться, может я что не так делаю.

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

Вот собственно сам проект: http://imperata.by/fiesta/

 

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

[Ironimus]

Может я чего то не понимаю, но зачем футеру давать класс footer?

[Killerblade]

Может я чего то не понимаю, но зачем футеру давать класс footer?

Что бы проще было выцепить самый главный футер из всех.

[afdw]

Кнопка "Найти" заезжает на "Поиск товара", особенно это заметно, если сфокусироваться на этом поле. Считаю, так будет покрасивее:

.header .header-section .header-aside .search .search-submit {border-top-left-radius: 0px;border-bottom-left-radius: 0px;}

Другой вариант:

 

.header .header-section .header-aside .search .search-submit {top: 4px;right: 4px;height: 29px;border-radius: 4px;}

[Killerblade]

 

Кнопка "Найти" заезжает на "Поиск товара", особенно это заметно, если сфокусироваться на этом поле. Считаю, так будет покрасивее:

.header .header-section .header-aside .search .search-submit {border-top-left-radius: 0px;border-bottom-left-radius: 0px;}

Другой вариант:

 

.header .header-section .header-aside .search .search-submit {top: 4px;right: 4px;height: 29px;border-radius: 4px;}

Согласен, так будет красивеет. Но дизайн, шаг в право, шаг в лево - растрел ! На щет того что в firefox немного не стыкуется при фокусе на инпут, в курсе. Еще раз спасибо.

[xzarxzes]

Всем привет.

 

Верстаю в принипе давно, но вот сейчас хотел посоветоваться, может я что не так делаю.

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

Вот собственно сам проект: http://imperata.by/fiesta/

 

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

А вы заголовками не пользуетесь, вы решили все div заменить на section ?

[Killerblade]

А вы заголовками не пользуетесь, вы решили все div заменить на section ?

Спасибо, я ждал этого вопроса. Да я решил заменить их всех на section. Только правда не понимаю, правильно ли я делаю. Валидатор конечно ругается что нету заголовочных тегов. Тут та я и решил поинтересоваться у вас, может знает кто как задобрить валидатор с этой ошибкой.

 

Вот результат валидации. http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fimperata.by%2Ffiesta%2F

Edited June 2, 2014 by Killerblade

[xzarxzes]

 

А вы заголовками не пользуетесь, вы решили все div заменить на section ?

Спасибо, я ждал этого вопроса. Да я решил заменить их всех на section. Только правда не понимаю, правильно ли я делаю. Валидатор конечно ругается что нету заголовочных тегов. Тут та я и решил поинтересоваться у вас, может знает кто как задобрить валидатор с этой ошибкой.

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

 

Лично мне кажется, вы решили сверстать на 100% html 5, не зная тольком что означают эти новые теги. Вы либо верстайте дивами, либо разберитесь оснавательно для чего нужны article, section итд.

Edited June 2, 2014 by xzarxzes

[Killerblade]

 

Лично мне кажется вы решили сверстать на 100% html 5, не зная тольком что означают эти новые теги. Вы либо верстайте дивами, либо разберитесь оснавательно для чего нужны article, section итд.

 

 

Да.. есть такое. Только я думаю что для тега footer я не смог бы пободрать нужных тегов из набора html5

<footer class="footer">  <section class="footer-section">    <article class="copyright">Fiesta-Viva.by 2014</article>    <article class="address">Адрес: г.Брест ул.Карьерная 12</article>    <article class="time">Время работы: 9:00 - 20:00 Без выходных</article>  </section></footer>

Преположим переделать этот блок.

 

В таком варианте мы заменяем абзацовый тег article на тот же абзац p

<footer class="footer">  <section class="footer-section">    <p class="copyright">Fiesta-Viva.by 2014</p>    <p class="address">Адрес: г.Брест ул.Карьерная 12</p>    <p class="time">Время работы: 9:00 - 20:00 Без выходных</p>  </section></footer>

Но не решенный варинат становится с .footer-section

Edited June 2, 2014 by Killerblade

[xzarxzes]

 

 

Лично мне кажется вы решили сверстать на 100% html 5, не зная тольком что означают эти новые теги. Вы либо верстайте дивами, либо разберитесь оснавательно для чего нужны article, section итд.

 

 

Да.. есть такое. Только я думаю что для тега footer я не смог бы пободрать нужных тегов из набора html5

<footer class="footer">  <section class="footer-section">    <article class="copyright">Fiesta-Viva.by 2014</article>    <article class="address">Адрес: г.Брест ул.Карьерная 12</article>    <article class="time">Время работы: 9:00 - 20:00 Без выходных</article>  </section></footer>

Преположим переделать этот блок.

 

В таком варианте мы заменяем абзацовый тег article на тот же абзац p

<footer class="footer">  <section class="footer-section">    <p class="copyright">Fiesta-Viva.by 2014</p>    <p class="address">Адрес: г.Брест ул.Карьерная 12</p>    <p class="time">Время работы: 9:00 - 20:00 Без выходных</p>  </section></footer>

Но не решенный варинат становится с .footer-section

 

<footer class="footer">   <small class="copyright">Fiesta-Viva.by 2014</small>   <span class="address">Адрес: г.Брест ул.Карьерная 12</span>   <span class="time">Время работы: 9:00 - 20:00 Без выходных</span></footer>

Ну допустим так, но можно сделать еще лучше использую микродату http://schema.org/address, http://schema.org/openingHours

[Killerblade]

<footer class="footer">   <small class="copyright">Fiesta-Viva.by 2014</small>   <span class="address">Адрес: г.Брест ул.Карьерная 12</span>   <span class="time">Время работы: 9:00 - 20:00 Без выходных</span></footer>

Ну допустим так, но можно сделать еще лучше использую микродату http://schema.org/address, http://schema.org/openingHours

 

 

Такой вопрос по разметке. В приведенном примере, у нас .footer выступает блоком во всю ширину, но нам надо разместить элементы о центру с огранничиной шириной (допустим 960). Как нам это сделать ?

[xzarxzes]

Такой вопрос по разметке. В приведенном примере, у нас .footer выступает блоком во всю ширину, но нам надо разместить элементы о центру с огранничиной шириной (допустим 960). Как нам это сделать ?

Обернуть в div, внутри футера надо сделать div контейнер

Edited June 2, 2014 by xzarxzes

[Killerblade]

Обернуть в div, внутри футера надо сделать div контейнер

Вот почему div. Почему не section, article ?

Edited June 3, 2014 by Killerblade

[xzarxzes]

 

Обернуть в div, внутри футера надо сделать div контейнер

Вот почему div. Почему не section, article ?

 

http://habrahabr.ru/post/124993/, https://vimeo.com/25823931

Edited June 3, 2014 by xzarxzes