Jump to content

Оцените верстку


Kisa1993
 Share

Recommended Posts

Беглый взгляд:

  1. При коротком доктайпе html кодировка объявляется просто <meta charset="кодировка" /> . И почему cp1251? Есть же utf-8.
  2. Что за глупости с айдишниками и собственными бекграундами для каждого пункта меню?
  3. По каким соображением практически везде используются ID?
  4. Что за глупости с кнопкой "SIGN UP" в виде img и JS для смены src по ховеру? Можно же просто сделать ссылку с текстом.
  5. Та же ерунда с лого. Можно менять по ховеру бекграунд, зачем там JS?
  6. div#header - это не хедер. Хедером у вас является див с непонятным классом roof100. header100 из той же оперы.
  7. Зачем дублировать фон у header, если у его родителя - такой же фон?
  8. width:100% для блочных элементов не нужно, за редким исключением.
  9. <div class="space"/> - что это? :)) margin-top/padding-top уже не работают?
  10. В html5 для группировки заголовков можно использовать hgroup (div.block>h3+h4)
  11. <div class="space_for_blocks"/> ?!
  12. Дублируется атрибут class: <img class="four_img" class="four_img" src="img/man.png"
  13. Неправильно объявлены комментарии. Комментарий закрывается БЕЗ восклицательного знака: <!-- комментарий -->
  14. В футере почему-то <p> обернут в <strong>

  • Like 2
Link to comment
Share on other sites

у тебя в верхнем меню сейчас 5 пунктов. представь, что на рабочем сайте добавлено еще 5. что случится, как думаешь?

Получится не очень хорошо :) . Но изначально в макете планировалось 5 пунктов).

Беглый взгляд:

1 Возьму на заметку, к utf-8 у меня какое-то отторжение, часто с ней мучилась, ну и есть крупные сайты которые на cp-1251.

2 У всех пунктов меню разный размер текста => и бэкграунд должен быть разным (на форуме писали что не такое уж и горе использовать и class и id вместе)

3 Не могу отучиться :angry: (хотя на форуме гуру писал что id для java а class для css)

4,5 Точно, признаю глупость)

6 header точно, не там использовала. А вот почему header100 непонятен?(

7 тупанула

8 не понимаю

9 упс :facepalmxd:

10 html 5 вообще не знаю и боюсь на него переходить (не знаю с чего начать)

11 упс :facepalmxd:

12 ощепятка

13 приму к сведению

14 а нужно только текст без тега(

Edited by Kisa1993
Link to comment
Share on other sites

1 Возьму на заметку, к utf-8 у меня какое-то отторжение, часто с ней мучилась, ну и есть крупные сайты которые на cp-1251.

По-моему, утф8 уже стала стандартом де-факто. Я не агитирую, но это очень удобная кодировка.

2 У всех пунктов меню разный размер текста => и бэкграунд должен быть разным (на форуме писали что не такое уж и горе использовать и class и id вместе)

В вашем случае можно использовать паддинги и border-radius. Если понадобятся картинки - можно воспользоваться "технологией раздвижных дверей": длинный бекграунд на li, и короткий (с другой стороны) - на <a>. В результате менюшка как бы раздвигается при увеличении текста

14 а нужно только текст без тега(

По поводу очистки флоатов: <div class="clearing"> и тому подобное - это уже история. Сейчас используют overflow:hidden родителю или clear:both в :after, погуглите на эту тему. На css-live.ru вывели новый экзотический способ, но я им не пользовался :)

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy