Прощу оценить верстку интернет магазина

[parnas]

Данный проект, это один из первых заказов в фрилансе. Старался последовать методологию MCSS. Хотя не думаю что смог реализовать методологию полностью. Буду дальше развиваться. Использовал SASS. Оцените пожалуйста качество верстки и поделитесь общем впечатлением. Побольше бы конструктивных критики. Заодно помогите с мелкими проблемами:

 

1) В хедере и футере есть пункт меню "О компании". При наведении просвечивается желтый фон и меняется цвет иконки. По сетке логотип и иконка(звездочка) должны быть на одном линии. Так и есть. Но при наведении должно быть padding между левой границой фона и иконкой. Часть фона может выйти за пределы центровшика. Это допустимо . Но эту фичу не смог реализовать. Как можно это делать?

 

2.В странице товара есть пункт Stonex S10 III GNSS / GSM / GPS / Wi-Fi / Bluetooth / CDMA / NETWORK"(radio) Когда часть текста переходить на следующую строку вес тексть не выравнивается на одной линии. Часть текста располагается под radio. как можно делать так что бы текст располагался на одном линии несмотря на переносы строк.

 

Главная, Каталог, Страница товара, Корзина 

 

Спасибо!

[Sergik+]

В каком браузере просвечивает ? У меня все нормально под виндой и лисой.

[parnas]

В каком браузере просвечивает ? У меня все нормально под виндой и лисой.

кажется неправильно выразился. Точнее, при ховере на пункт появляеться желтый фон  со стрелкой. Это нормально. Просто у первого пункта "О компании" не получаеться никак задать левый отступ от иконки. Попробуйте навести мышку другим пунктам. Там есть этот отступ. А у первого пункта фон прям прилип к иконку.

 

[Sergik+]

Угу при hover нету padding-left.

Так у вас layout-center group обертка не дает левому паддингу место.

Я думаю надо дать всему меню левый отступ чуть.

Edited August 26, 2014 by Sergik+

[Sergik+]

Смотрю вашу верстку и есть вопрос а не замечание совсем, нужно ли каждому элементу

давать класс ? Как раз смотрю выступление Вадима Макеева по БЭМ так он там говорит

что надо под себя подстраивать любую методологию разметки css.

Я сам еще только начал изучать данную тему и вопросов очень много.

[parnas]

Угу при hover нету padding-left.

Так у вас layout-center group обертка не дает левому паддингу место.

Я думаю надо дать всему меню левый отступ чуть.

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

Смотрю вашу верстку и есть вопрос а не замечание совсем, нужно ли каждому элементу

давать класс ? Как раз смотрю выступление Вадима Макеева по БЭМ так он там говорит

что надо под себя подстраивать любую методологию разметки css.

Я сам еще только начал изучать данную тему и вопросов очень много.

Да , в каком-то степени вы правы. Я тоже только начал изучать, внедрять методологии. Но сам Роберт Харитонов на презентации тоже подчеркнул что-то не нужно бояться подкорректировать методологии под себя и это не противоречить принципам методологии. Хотя, конечно, есть вещи которых не рекомендуется модифицировать.

Edited August 26, 2014 by parnas

[Sergik+]

Почитайте здесь две верхние статьи http://web-standards.ru/ , очень интересные точки зрения.