Оцените написание кода(крохотный кусочек работы)

[KamZed]

Всем доброго времени суток, пожалуйста посмотрите на код(небольшой кусочек работы) и оцените, хотелось бы услышать что правильно что нет, что можно было бы по другому, как можно детальней и дотошнней, семантика пространство имен и прочее прочее, в общем чем больше критики(желательно конструктивной и обоснованной) тем лучше, я специально выкладываю маленький кусок что бы, было не напряжно  его изучить!

http://codepen.io/Kamzed/pen/rrqJJz

Заранее благодарю всех добрых людей которые согласятся помочь. 

 

[wwt]

* {
	box-sizing: border-box;
}

по рукам за это уже.

[Нарек]

10 часов назад, wwt сказал:

* {
	box-sizing: border-box;
}

по рукам за это уже.

Разработчиков бутстрапа надо еще пинать за это.

[d0ublezer0]

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

по рукам за это уже.

Почему?

[wwt]

4 минуты назад, d0ublezer0 сказал:

Почему?

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

[alejandro13]

 

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

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

И как правильнее будет записать это правило?

[wwt]

4 минуты назад, alejandro13 сказал:

 

И как правильнее будет записать это правило?

прописать это свойство только тем элементам(классам, тегам и т.д.) которым нужно, это же очевидно.

[jokerinteractive]

1. для 

.index-header {}

не стоит задавать высоту, либо min-height, либо отступами (это элементарная защита от переполнения контентом)

2. зачем лишний блок делать `.upper-header`? Градиент можно сразу к `.index-header` селектору применить.

3. Лишняя обёртка `.index-logo > h1`, можно сразу `h1.index-logo`, а лучше сразу `h1.logo`, а ещё лучше не использовать заголовок первого уровня для логотипа (для СЕО не очень), а делать ссылкой на главную страницу, люди привыкли, что логотип — ссылка на главную, деже на главной (для перезагрузки страницы, например) `a.logo` Наилучший вариант я считаю.

4. Зачем опять лишняя обёртка `div.user-block`? Что мешает делать сразу `a.user-block`?

5. По спеке в форме обязательный атрибут `action` должен быть

6. В форме вместо кнопке добавить `type="submit"` для отправки формы ентером

7. ну и про БЭМ имеет смысл почитать)) 

В целом — норм, молодец!) Продолжай практиковаться и у тебя всё получится.

[KamZed]

В 19.10.2016 в 17:54, jokerinteractive сказал:

 

В целом — норм, молодец!) Продолжай практиковаться и у тебя всё получится.

Спасибо большое  за советы, я это очень ценю!

[andrey7287]

В 19.10.2016 в 10:09, wwt сказал:

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

И что в этом криминального?

[Николя223]

1 час назад, andrey7287 сказал:

И что в этом криминального?

когда мощности компов были не такими как сейчас ходили слухи что это правило тормозило рендеринг страницы.

да и лучше использовать для сброса специально подготовленные нормализаторы стилей

 

[wwt]

1 час назад, andrey7287 сказал:

И что в этом криминального?

Криминального в этом нет. Вопрос не в криминале, если рассуждать просто то это правило вполне себе правило, а если рассуждать с точки зрения целесообразности то становится очевидно что в таком правиле нет необходимости.

29 минут назад, Николя223 сказал:

когда мощности компов были не такими как сейчас ходили слухи что это правило тормозило рендеринг страницы.

да и лучше использовать для сброса специально подготовленные нормализаторы стилей

 

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

[andrey7287]

18 часов назад, Николя223 сказал:

когда мощности компов были не такими как сейчас ходили слухи что это правило тормозило рендеринг страницы.

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

18 часов назад, wwt сказал:

Криминального в этом нет. Вопрос не в криминале, если рассуждать просто то это правило вполне себе правило, а если рассуждать с точки зрения целесообразности то становится очевидно что в таком правиле нет необходимости.

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