Jump to content

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

alexforce2

By alexforce2,
in Discussion of works

 Share

Recommended Posts

alexforce2 Explorer

alexforce2

Posted
Posted

Всем привет. Сверстал очередной макет для тренировки, прошу оценить код хтмл и цсс, а также хотелось бы услышать мнения о нарезанных изображениях(хорошо,плохо, что можно было сделать лучше). _http://alexforce2.6te.net/

С удовольствием выслушаю конструктивную критику.

Link to comment
Share on other sites
br3t Explorer

br3t

Posted
Posted

- если вы доктайпитесь как НТМЛ5 - используйте его (header, footer etc)

- думаю, нет смысла загружать три скрина отдельно, только если вы не собираетесь их ротировать

- вы используете box-shadow на кнопках (чёрная тень на чёрном фоне не видна), но почему-то игнорируете border-radius на тех же кнопках. И меня смущает кнопка, сделанная параграфом

- иконки в фиче-листе стОит брать в png-24, на спасательном круге особо заметен jpegный "жир"

- не пренебрегайте использованием ul-li

Link to comment
Share on other sites
alexforce2 Explorer

alexforce2

Posted
  • Author
Posted

- насчёт трёх скринов - сделал три штуки на случай если придётся немного увеличить блок по высоте.

- про какие box-shadow на кнопках с чёрным фоном вы говорите? вроде кнопок не делал. border-radius к красным блокам на чёрном фоне не стал применять потому что оно не поддерживается старыми ослами, да и целиком фоном применить легче. Но у этого способа есть минус - при изменении кол-ва текста, он выйдет за границы.

Большое спасибо br3t!

Кто может - покритикуйте за код.

Link to comment
Share on other sites
Vin Explorer

Vin

Posted
Posted (edited)

- если вы доктайпитесь как НТМЛ5 - используйте его (header, footer etc)

Необязательно. <!DOCTYPE html> кроссбраузерен, в отличии от новых элементов html5 (хотя и поддержка лечится небольшим js-костылём, но всё же).

- про какие box-shadow на кнопках с чёрным фоном вы говорите?

"Shop Indvidual Templates Starting at Just $25" и "Join the Club! Get Access to All Templates for $49". Посмотрите повнимательнее на макете — там есть тени, отбрасываемые текстом. Можно сделать свойством text-shadow, в старых ослах будет выглядеть вполне хорошо (как сейчас), просто в нормальных браузерах будет выглядеть лучше.

- hover'ы желательно как-то обозначить. Лучше всего, имхо, задать им цвет #fff.

- Логотип практически всегда делается картинкой.

Edited by Vin
Link to comment
Share on other sites
Vin Explorer

Vin

Posted
Posted

А можно фон кнопки и текст слить и вставить картинкой? он (текст) по сути никакой ценности не представляет же.

на днях закончил верстать этот же шаблон и постеснялся выложить

Всё зависит от ТЗ, которое, в свою очередь, зависит от конкретной ситуации (как будет организована поддержка сайта).

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
Go to topic listing
×
×
  • 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

  I accept