Jump to content

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


aknur
 Share

Recommended Posts

Здравствуйте!

Сверстал psd макет скаченный на просторах интернета.

Можете пожалуйста оценить верстку и указать на ошибки и.т.п.

Буду очень очень благодарен!

СКАЧАТЬ С PSD

———--

СКАЧАТЬ БЕЗ PSD

Link to comment
Share on other sites

Ошибки валидатор скажет. Вначале надо исправить явные ляпы, тогда уже можно и обсуждать.

Валидатором ошибки исправил.

закрыл все теги img, убрать <p> который мешался и теперь все валидно.

Проверьте код дальше пожалуйста

Link to comment
Share on other sites

onmouseover для логотипа использовать не обязательно, удобнее через hover сделать. Саму картинку придется фоном положить и лучше использовать спрайты (объединить две картинки в одну).

@charset "utf-8"; - в стилях эта строка не нужна, кроме латиницы других символов нет.

* {

padding:0;

margin:0;} - вот это осуждаю и свою позицию буду отстаивать.

Link to comment
Share on other sites

onmouseover для логотипа использовать не обязательно, удобнее через hover сделать. Саму картинку придется фоном положить и лучше использовать спрайты (объединить две картинки в одну).

@charset "utf-8"; - в стилях эта строка не нужна, кроме латиницы других символов нет.

* {

padding:0;

margin:0;} - вот это осуждаю и свою позицию буду отстаивать.

всмысле это плохо ?

Link to comment
Share on other sites

Зачем вместо него что-то добавлять? Где надо обнулить отступы, обнуляешь, * {} выкидываешь из стилей.

Мне кажется, что проще добавить отступы где нужно, чем обнулять в отдельно взятых блоках.

И нельзя говорить, что глобальное обнуление это плохо. Оно плохо исключительно для тебя и мнение является субъективным.

Кому как удобнее, тот так и делает.

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

  • Обсуждения

    • Актуальные контакты: 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