Jump to content

Оцените качество вёрстки


weband
 Share

Recommended Posts

-Ссылки никак себя не ведут при наведение. Абсолютно..


<ul id="menu_list">
<li class="first">

-Зачем для этого отдельный класс? Воспользуйтесь псевдоклассом first-child к примеру.

-Вы фанат таблиц? Очень многое сделано с их помощью..Но стоит ли оно того? Точнее даже нужны ли?


#resort_search input#input_first {

-ID уникален, зачем тогда контекстный селектор?

-Хорошим тоном считается прижатие футера к низу.

-В футере можно было разделитель меню выровнять и точнее :)


/* @import url(reset.css); */
* {margin:0; padding:0; border:0; outline: 0;}

-Лучше бы закомментировали 2ю строчку :)

Edited by Vulner
Link to comment
Share on other sites

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

Вроде вёрстка нормальная (я не профи,мне не судить,но явных ошибок,которые я бы смог заменит,вроде нет.). Хотлесь бы что бы помимо фона wrapper был и общий фон,а то скудно смотрится "полоска с информацией" на белом фоне .

Так же думаю лучше меню в футоре придать к низу или по середине (хотя на шаблоне он прибит к верху),но я если вы делаете для себя посмотрите,если прибить к низу или к середине может лучше выглядит ?

На макете кнопка в форме сделана в виде градиента (нет,цвет тоже не плохо и это конечно мелочи,но можно писать на css3 или с помощью генераторов градиента),а у вас в цвете.

Расскажите как обойтись без таблиц?

Блочной вёрсткой =)

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

  • Similar Content

    • By ZondArt
      Здравствуйте. Страницу сверстал из бесплатного PSD-шаблона. Хотелось бы услышать критику по поводу качества вёрстки, а также по поводу javascript кода. JS писал сам, единственное для удобства работы с DOM'ом использовал jQuery.
      Страница: http://artemkajs.meximas.com/
      jpeg шаблона: http://higgs.rghost.ru/53507945/image.png
      psd: http://rghost.ru/53508127
    • By Teosar1
      Здравствуйте, прошу покритиковать мою вёрстку. Html5 пока не трогаю. Id применяю.
       
      http://testx.96.lt/index2.html
  • Обсуждения

    • Актуальные контакты: Telegram: @Nikker_web E-Mail:   tarasevich.email@gmail.com Портфолио https://www.behance.net/d4d4186e Разрабатываю дизайн групп в соц сетях, сайтов, приложений, другой дизайн под заказ    
    • Актуальные контакты: Telegram: @Nikker_web E-Mail:   tarasevich.email@gmail.com   Разрабатываю дизайн групп в соц сетях, сайтов, приложений, другой дизайн под заказ   Портфолио https://www.behance.net/d4d4186e
    • Доброго всем времени суток. Прошу помощи. Научите принципу изменения футера. Движок 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 } }  
    • Нужны сайты с примерами верстки, типа https://css-tricks.com/. Типовые приемы и нестандартные на все случаи жизни. Накидайте ссылок.
×
×
  • 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