Jump to content

First work


hellpirat
 Share

Recommended Posts

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

 

http://learnhtm-css.site11.com/

 

http://savepic.net/4929448.jpg

 

:(  спасибо

 

Link to comment
Share on other sites

Тут оценивать, собственно, нечего. Полное несоответствие макету. Зачем вы выкладываете не законченную работу на обсуждение?

 

Из того, что бросилось в глаза:

Зачем у body задали overflow: scroll?

Все содержимое body нужно положить в общий контейнер и отцентрировать.

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

Link to comment
Share on other sites

Тут оценивать, собственно, нечего. Полное несоответствие макету. Зачем вы выкладываете не законченную работу на обсуждение?

 

Из того, что бросилось в глаза:

Зачем у body задали overflow: scroll?

Все содержимое body нужно положить в общий контейнер и отцентрировать.

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

Да я понимаю, что я не доделал...Но я сделал все, что было в моих силах,хотелось бы увидеть ссылки на какие нибудь ресурсы(книги,курсы) которые помогут мне исправить все мои ошибки и улучшить верстку 

Link to comment
Share on other sites

 

Тут оценивать, собственно, нечего. Полное несоответствие макету. Зачем вы выкладываете не законченную работу на обсуждение?

 

Из того, что бросилось в глаза:

Зачем у body задали overflow: scroll?

Все содержимое body нужно положить в общий контейнер и отцентрировать.

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

Да я понимаю, что я не доделал...Но я сделал все, что было в моих силах,хотелось бы увидеть ссылки на какие нибудь ресурсы(книги,курсы) которые помогут мне исправить все мои ошибки и улучшить верстку 

 

не используй br для отступов, у тебя br в вёрстке зависят от размера шрифта, это всё вводит хаус, лучше пропиши padding, margin

2) шапка, зачем ты её забил в список ? в этом нет нужды, при этом используешь флоат

используй clearfix

3) меню зачем ты все элементы вставил одной картинкой, 

m1.png

 

советую сначала посмотреть как свёрстаны похожие сайты, там и подчепнёшь как надо

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