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

[stan07]

Добрый день!

Оцените резиновую верстку, укажите на ошибка, пожалуйста.

http://weburoki.com.ua/roof/

[npofopr]

Резиновость?

Под резиновостью хотелось увидеть некую адаптивность что ли. А тут всё заканчивается всё на 1060px Печально, даже не на 1024px.

[klierik]

в целом не плохо...

но:

1. научится писать без комментариев

2. скорее всего не bunner а banner

3. скорее всего не brends а brands

4. в блоке .news меня смущают <artice>. то больше похоже на список новостей

5. блок отзывов так же скорее является списком

6. избавися от транслита в именах классов

<div class="zakaz"></div>

<ul class="products_spisok">

7. научится именовать классы так что бы по названию было понятно что это за элемент

<div class="inner_spec_pred_left">

8. и это тоже список:

<div class="foto_galery clearfix">

9. блок

<ul class="soc_seti">

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

и добавить ссылкам содержимое в title=""

10. скрипты перенести в область перед закрывающим </body> -- ну то такое...

на закуску пример форматирования стилей что бы они были удобочитаемые: http://screencast.com/t/2W6tb8OQ5Du

[Bassline]

1. научится писать без комментариев

Может быть я чего-то недопонимаю, но почему вы так упорно советуете не комментировать, тогда как на вашем личном сайте и на проектах представленных в нем, комментарии содержатся как в html, так и в css документах буквально на каждой строчке?

Edited June 13, 2013 by Bassline

[stan07]

Резиновость?

Под резиновостью хотелось увидеть некую адаптивность что ли.

Адаптивная верстка это мой следующий шаг)

А тут всё заканчивается всё на 1060px Печально, даже не на 1024px.

Странно min-width: 1024px стоит эт наверно из за скрола в браузере

на закуску пример форматирования стилей что бы они были удобочитаемые: http://screencast.com/t/2W6tb8OQ5Du

Я считаю что это дело вкуса)

[klierik]

Я считаю что это дело вкуса)

А я, опираясь на опыт, считаю что когда у тебя 30-50 CSS фалов в проекте и в самом маленьком файле более 500 строк, то таким образом поддерживать, читать, управлять, понимать код можно быстрее, качественее, удобнее.

Хотя к полезным привычкам тебя никто не заставляет привыкать

Ведь ты врядли думаешь, во время написания кода, о том что после тебя кто-то проект будет вести дальше

[stan07]

Я считаю что это дело вкуса)

А я, опираясь на опыт, считаю что когда у тебя 30-50 CSS фалов в проекте и в самом маленьком файле более 500 строк, то таким образом поддерживать, читать, управлять, понимать код можно быстрее, качественее, удобнее.

Хотя к полезным привычкам тебя никто не заставляет привыкать

Ведь ты врядли думаешь, во время написания кода, о том что после тебя кто-то проект будет вести дальше

Не знаю может вы и правы, но, почему-то , в большинстве версток которых я видел , больших и маленьких проектах , CSS пишут в столбик

[klierik]

Не знаю может вы и правы, но, почему-то , в большинстве версток которых я видел , больших и маленьких проектах , CSS пишут в столбик

Большинство не приделяют этому значние потому что не имеют опыта работы в больших проектах. Их работа, как правило, содержится в рамках создания сайток-визиток и в рамках сложности выше не растёт.

Такой процесс описывается как: Получить задание -> сделать -> сдать -> забыть. Все остальные проблемы исполнителя касатся не будут и все проблемы лягут на плечи другого разработчика.

Уж поверь, когда тебе прийдется доделывать/обновлять такой проект через 1-2 года (даже если его написал ты), ты будешь проклинать того кто этот код написал

[stan07]

Не знаю может вы и правы, но, почему-то , в большинстве версток которых я видел , больших и маленьких проектах , CSS пишут в столбик

Большинство не приделяют этому значние потому что не имеют опыта работы в больших проектах. Их работа, как правило, содержится в рамках создания сайток-визиток и в рамках сложности выше не растёт.

Такой процесс описывается как: Получить задание -> сделать -> сдать -> забыть. Все остальные проблемы исполнителя касатся не будут и все проблемы лягут на плечи другого разработчика.

Уж поверь, когда тебе прийдется доделывать/обновлять такой проект через 1-2 года (даже если его написал ты), ты будешь проклинать того кто этот код написал

обязательно попробую, спасибо

[Kray Storm]

пример форматирования стилей

Надо же! Как раз на днях экспериментировал с форматированием стилей, из расчета под "много запутанного кода и классов". И как раз пришел к такому же решению.

Намного удобней классического "столбика". А если еще делать разбивку комментариями на структурные блоки с одинаковым наименованием в html и css, то читаемость возрастает заметно.

Задавать, условно, "всем и вся" классы только поначалу казалось нелогично. Ну да... ведь у нас в CSS есть крутое каскадирование на любой вкус. Но когда вдруг приходится полностью переместить какой-то блок, вставить внутрь, одной сложной конструкции элементов - другую, или поменять теги и при этом стили совершенно не нужно трогать - этот подход быстро оцениваешь.

И длинные имена классов, вроде .head-logo-slogan-text{...;} хоть поначалу и пугают, но на деле намного быстрее читаешь стили, чем с привычными ранее div div p span.text{...;}.

На счет "писать без комментариев" - оно конечно, приучает к красивому коду. Но когда есть завязанные друг с другом значения (например, высота одного блока, отступ другого, line-height у строки внутри), то помогает у определяющего значения пометить "там-то и там-то завязка на это число". Сразу становиться понятно, где нужно делать изменения, если что. Ну это сугубо любительское мнение, у меня опыта не так много.

[Int]

Увеличение маленьких картинок после 1060px удручает, мыло на мыле.

Чтоб не спорить об именах классов: в них можно использовать любые символы UTF же, пиши названия по-русски и всего делов!

[stan07]

Чтоб не спорить об именах классов: в них можно использовать любые символы UTF же, пиши названия по-русски и всего делов!

Спасибо, буду знать )