Владимир Ш.
25 повідомлень
#16 років тому
Решил попробовать свои силы в верстке и сверстал бесплатный шаблон. Буду благодарен за критику и полезные советы.
Адрес:

Как в целом читаемость и простота кода?
Руслан Савенок
530 повідомлень
#16 років тому
На первый взгляд все нормально, вникать не хочеться.

Зачем писать класс png можно в ксс сделать для всех картинок бихавиор

И графику сожмите
Владимир Ш.
25 повідомлень
#16 років тому
Цитата ("zombieman"):


Зачем писать класс png можно в ксс сделать для всех картинок бихавиор



А можно примерчик?
В скриптах разбираюсь не очень. Этот позаимствовал у студии Лебедева.
Михаил В.
2195 повідомлень
#16 років тому


New recipe вылезает из блока, заметно.

class="last" в меню стоит избегать, программисту может быть неудобно. Вариантов сделать разделители в меню без лишних классов много, например:

<!--><!><!--> - слишком много таких вставок в теле страницы, весьма неудобно при интеграции и отладке. Даже адепты подобного подхода как Stu Nichols отходят от использования CC вне <head>.

Можно было сверстать без лишних элементов типа cross/draw/bord.
<div class="clearfix"></div> - тоже мусор, есть способы сброса float без подобных лишних вставок.

Злоупотребляете id, на внутренних возможно какие-то блоки могут потребоваться повторно.

CSS вполне читабелен. Немного непонятен порядок свойств, в разных id разный порядок встречается.
Комментарии в самом html коде по большому счету необязательны, большинство толковых программистов используют редакторы/IDE с поддержкой фолдинга тегов.

Кроссбраузерность не смотрел.
Ну и макет не самый приятный на глаз вы выбрали для верстки.

А class="png" я и не заметил.
Владимир Ш.
25 повідомлень
#16 років тому
Цитата ("Sir_Michael"):
New recipe вылезает из блока, заметно.

А можно уточнить браузер?
У меня, кажется не вылезает.

Цитата ("Sir_Michael"):
class="last" в меню стоит избегать, программисту может быть неудобно. Вариантов сделать разделители в меню без лишних классов много, например:

<!--><!><!--> - слишком много таких вставок в теле страницы, весьма неудобно при интеграции и отладке. Даже адепты подобного подхода как Stu Nichols отходят от использования CC вне <head>.

учту.

Цитата ("Sir_Michael"):

Можно было сверстать без лишних элементов типа cross/draw/bord.
<div class="clearfix"></div> - тоже мусор, есть способы сброса float без подобных лишних вставок.

Другие варианты пробовал, но не подошли.
В отношении float знаю еще об overflow:hidden и float родителю. Но в первом случае не получается задать фон выходящий за пределы блока, во втором случае - родительский блок не влияет на другие, так как выпадает из потока.

Цитата ("Sir_Michael"):
Злоупотребляете id, на внутренних возможно какие-то блоки могут потребоваться повторно.

CSS вполне читабелен. Немного непонятен порядок свойств, в разных id разный порядок встречается.
Комментарии в самом html коде по большому счету необязательны, большинство толковых программистов используют редакторы/IDE с поддержкой фолдинга тегов.

учту.

Цитата ("Sir_Michael"):
Кроссбраузерность не смотрел.

Но с этим собственно и связаны условные комментарии в теле. Если сказать коротко: по другому не мог решить проблему с z-index в ИЕ 6 и 7, связанную с ненужным образование стекового контекста во всех позиционированных элементах.