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

  • Обсуждения

    • Nikker
      Дизайн сайтов, landing page, логотипов, баннеров, шапок | Высокое качество, по хорошей цене

      By Nikker · Posted

      Актуальные контакты: Telegram: @Nikker_web E-Mail:   tarasevich.email@gmail.com Портфолио https://www.behance.net/d4d4186e Разрабатываю дизайн групп в соц сетях, сайтов, приложений, другой дизайн под заказ    
    • Nikker
      Брендовый дизайн в социальных сетях

      By Nikker · Posted

      Актуальные контакты: Telegram: @Nikker_web E-Mail:   tarasevich.email@gmail.com   Разрабатываю дизайн групп в соц сетях, сайтов, приложений, другой дизайн под заказ   Портфолио https://www.behance.net/d4d4186e
    • Mondeus
      Вёрстка футера на движке Xenforo

      By Mondeus · Posted

      Доброго всем времени суток. Прошу помощи. Научите принципу изменения футера. Движок Xenforo. Версия 2.2.10. Стиль дефолтный. Что именно нужно в итоге на фото примере. Мой шаблон app.footer less имеет следующее значение.  .p-footer { .xf-publicFooter(); a { .xf-publicFooterLink(); } } .p-footer-inner { .m-pageWidth(); .m-pageInset(); padding-top: @xf-paddingMedium; padding-bottom: @xf-paddingLarge; } .p-footer-row { .m-clearFix(); margin-bottom: -@xf-paddingLarge; } .p-footer-row-main { float: left; margin-bottom: @xf-paddingLarge; } .p-footer-row-opposite { float: right; margin-bottom: @xf-paddingLarge; } .p-footer-linkList { .m-listPlain(); .m-clearFix(); > li { float: left; margin-right: .5em; &:last-child { margin-right: 0; } a { padding: 2px 4px; border-radius: @xf-borderRadiusSmall; &:hover { text-decoration: none; background-color: fade(@xf-publicFooterLink--color, 10%); } } } } .p-footer-rssLink { > span { position: relative; top: -1px; display: inline-block; width: 1.44em; height: 1.44em; line-height: 1.44em; text-align: center; font-size: .8em; background-color: #4682B4; border-radius: 2px; } .fa-rss { color: white; } } .p-footer-copyright { margin-top: @xf-elementSpacer; text-align: center; font-size: @xf-fontSizeSmallest; } .p-footer-debug { margin-top: @xf-paddingLarge; text-align: right; font-size: @xf-fontSizeSmallest; .pairs > dt { color: inherit; } } @media (max-width: @xf-responsiveMedium) { .p-footer-row-main, .p-footer-row-opposite { float: none; } .p-footer-copyright { text-align: left; padding: 0 4px; // aligns with other links } }  
    • torsar
      Примеры верстки

      By torsar · Posted

      Нужны сайты с примерами верстки, типа https://css-tricks.com/. Типовые приемы и нестандартные на все случаи жизни. Накидайте ссылок.
    • Katerina23
      Какой тег использовать, чтобы увеличивать число кнопками вверх-вниз?

      By Katerina23 · Posted

      Да, подходит. Спасибо.
×
×
  • 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