Оцените первую вёрстку по макету...

[manhunt]

Доброго времени суток! Не смог создать новую тему в соответствующем разделе, поэтому пишу здесь!!

Макет брал вот от сюда http://forum.htmlbook.ru/index.php?showtopic=16763

Ссылка на вёрстку http://test-manhunt.hdd1.ru/

Прокомментируйте пожалуйста вёрстку!!!!

[MC WEST]

Нууу-с начнем Во-первых не в ту категорию запостил. Дальше смотри список:

Визуальные проблемы (просматривал в браузере Chromium 10.0):

1. Пункты меню криво выровнянны.

2. Кнопка в форме поиска уехала непонятно куда.

3. Основной правый блок полностью разъехался, сотворив черти что на странице.

4. Мелочь: Нету рамок как в шаблоне.

5. У логотипа (ссылки) не убрано подчеркивание.

Проблемы по коду:

1. Что это? <link stylesheet=""> Ну как-бы я понимаю, что это, но зачем?

2. У Вас везде используется id, используйте class, оставьте id разработчикам (названия самих id меня иногда поражали).

3. Форму поиска не обязательно надо было помещать в еще один див.

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

А для первого раза, очень даже хорошо, учись и стремись узнавать все больше и больше.

[manhunt]

Спасибо за замечания!!!!

[cssview]

  On 4/4/2011 at 6:36 PM, MC WEST said:

Нууу-с начнем Во-первых не в ту категорию запостил. Дальше смотри список:

Визуальные проблемы (просматривал в браузере Chromium 10.0):

1. Пункты меню криво выровнянны.

2. Кнопка в форме поиска уехала непонятно куда.

3. Основной правый блок полностью разъехался, сотворив черти что на странице.

4. Мелочь: Нету рамок как в шаблоне.

5. У логотипа (ссылки) не убрано подчеркивание.

Проблемы по коду:

1. Что это? <link stylesheet=""> Ну как-бы я понимаю, что это, но зачем?

2. У Вас везде используется id, используйте class, оставьте id разработчикам (названия самих id меня иногда поражали).

3. Форму поиска не обязательно надо было помещать в еще один див.

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

А для первого раза, очень даже хорошо, учись и стремись узнавать все больше и больше.

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

В общем да, вид лучше чем код. Нужно hmtl полностью переработать.

Ну например, сделать изначально две колонки. Тег div#head перенести во вторую колонку. В первой колонке "домик" подложкой, логотип позиционировать ну и т.д.

Это как я бы делал. Ну, а так, учится и еще раз учится.

[Serlutin]

  Quote

Ну например, сделать изначально две колонки. Тег div#head перенести во вторую колонку.

Зачем? Если и так нормально? Наоборот, лучше когда шапка отделена от основной части, так прощё на движок насаживать. Шапка то везде одна, а вот информация в колонках будет меняться.

Жалко, что так часто position: absolute используется. Ведь этот дизайн можно сделать резиновым.