Jump to content
  • 0

Прошу вашей критики

fenix_63

Asked by fenix_63,

 Share

Question

fenix_63 Explorer

fenix_63

Posted
Posted (edited)

Народ, всем привет, в общем я только начал изучать верстку. Взял шаблон вот тут: http://www.graphicsfuel.com/wp-content/uploads/2010/09/corporatix.jpg  Хотел сделать резиновую верстку, но когда меняю размеры браузера, всё совсем плохо становится.

Разместил свои наработки вот тут: http://togliatty.ucoz.ru/   Это моя первая наработка.

Прошу опытных людей указать мне на мои недоработки, недочеты, грубые ошибки, и способ их исправления. Заранее благодарен.

Edited by fenix_63
Link to comment
Share on other sites

6 answers to this question

Recommended Posts

  • 0
swandev Explorer

swandev

Posted
Posted

Плохо, очень плохо.

1. Нужен доктайп <!doctype> http://tmlbook.ru/html/%21doctype

2. У <body> сбросить отступы, я рекомендую использовать normalize.css http://necolas.github.io/normalize.css/

3. Убрать  высоту в %-тах. на всех блоках, вообще всю резину убрать. Для начала научитесь делать обычную верстку, и сейчас больше актуальна адаптивная верстка.

 

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

  • Like 1
Link to comment
Share on other sites
  • 0
fenix_63 Explorer

fenix_63

Posted
  • Author
Posted

Понял, спс.  В общем я доктайп добавил, % убрал. Макет статичный стал. Слайдер такой потому что как я понял просто картинки другие нужны. У меня тоже была идея попробовать начать делать адаптивную верстку, но думаю надо начинать от простого к сложному.

 

Подскажите ещё ошибки

Link to comment
Share on other sites
  • 0
swandev Explorer

swandev

Posted
Posted

1. background логотипа не соответствует фону шапки.

2. id="menu"

- т.к это меню, то нужно добавить ссылки в тег <li>

- чтобы стилизовать активный пункт добавляем к тегу li класс .active и стилизуем через этот класс (#menu ul li.active a {})

- меню разваливается http://joxi.ru/nv8jVIwyTJDfXDAlZ-0

 

3. Убрать высоту там где должен изменяться контент, убрать полностью т.к высота должна формироваться в зависимости от контента внутри

 

4. Не соответствует дизайну, отступы, размер шрифтов и т.д

Link to comment
Share on other sites
  • 0
fenix_63 Explorer

fenix_63

Posted
  • Author
Posted

background исправил, ссылки в теги li добавил. Активный пункт меню пока стилизовать не стал, только небольшую подсветку для a:hover сделал.  Не совсем понятно ка реализовать размеры блоков в зависимости от наполнения контента. Размеры шрифтов если брать как в шаблоне, то не влезет на страницу всё, поэтому подгонял. Буду допиливать дальше )

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
Answer this question...

×   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 question 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