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

[oleg126]

Собственно вот - http://testsiteoleg126.tk/Environmental.com/

[amelice]

1. Не хорошо выставлять  контейнеру #wrapper высоту.

2. откуда вы нашли тег content?

3. вот это все лого http://i.imgur.com/oFWcMAZ.png, а не отдельные элементы.

Если вы не знаете как оформить какие-то элементы, откройте общеизвестные сайты, посмотрите как там оформлены отдельные элементы.
4. http://i.imgur.com/bh3QHtu.png - это обычно единый контент,  и картинка оформляется картинкой, а не бекграундом.  Обходится без абсолюта.
5. http://i.imgur.com/XoEtSAv.png - это кнопка, а не заголовок.

6. Много абсолютных элементов, там где не нужно.

7. Это очевидно список ссылок http://i.imgur.com/kPeRs18.png, а не абзац.
8. Присваиваете свойство  display: block элементам уровня блок- div, h... . Этого не нужно, браузер и так знает. 

Edited June 13, 2014 by amelice

[oleg126]

1. Не хорошо выставлять  контейнеру #wrapper высоту.

2. откуда вы нашли тег content?

3. вот это все лого http://i.imgur.com/oFWcMAZ.png, а не отдельные элементы.

Если вы не знаете как оформить какие-то элементы, откройте общеизвестные сайты, посмотрите как там оформлены отдельные элементы.

4. http://i.imgur.com/bh3QHtu.png - это обычно единый контент,  и картинка оформляется картинкой, а не бекграундом.  Обходится без абсолюта.

5. http://i.imgur.com/XoEtSAv.png - это кнопка, а не заголовок.

6. Много абсолютных элементов, там где не нужно.

7. Это очевидно список ссылок http://i.imgur.com/kPeRs18.png, а не абзац.

8. Присваиваете свойство  display: block элементам уровня блок- div, h... . Этого не нужно, браузер и так знает. 

 

1. Исправил.

2. Видимо придумал сам) Почему то казалось, что на фоне тегов header и footer должен быть и content.

3. Ок, учту.

4. Картинку вставить через img src="" ?

5. Исправлю.

6. Возможно из за того, что я хорошо знаю, как он работает, и иногда когда, не получается сделать иначе делаю им(

7. Мне показалось, что это просто подчеркнутый текст.

8. Например, вроде только при вставке картинок использую, но там вроде это надо!?

[amelice]

 

2. Видимо придумал сам) Почему то казалось, что на фоне тегов header и footer должен быть и content.

 

 

Хорошее у вас воображение 

Есть тег main для этого, но обязательно почитайте про него перед тем как его использовать, потому что есть требования по его использованию.

4. Да картинку через src, а есть альтернативы?  

В данном случае наверное да, потому что картинка простая. 

 

6. Возможно из за того, что я хорошо знаю, как он работает, и иногда когда, не получается сделать иначе делаю им(

6. Точно хорошо знаете?  Какой элемент является контейнером для абсолютного элемента? Если не получается спрашивайте тут, что конкретно не получается.

7. Вы ошибаетесь.

8.Вот здесь http://www.w3.org/TR/CSS2/sample.html представлены дефолтные стили для элементов (html4). Всем элементам, кто имеет свойство {display: block} не нужно присваивать его дополнительно.

 

Edited June 14, 2014 by amelice