Jump to content

Первая верстка в моей жизни.Прошу оценить.


Recommended Posts

Первопост.

Сайт — http://dimaspavlov.16mb.com/

Макет — https://yadi.sk/i/Cx8jTKfxt4anV

Прошу оценить раскритиковать верстку, начиная с HTML-структуры и указать на все недостатки. Так и не смог сделать переход для градиента в навигационном меню.

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

P.S. Верстка не адаптивная.

Edited by DmitryPavlov
Link to comment
Share on other sites

Для начала весьма недурно. Сразу бросается в глаза то, что верстка не гибкая, то есть при увеличение количества текста высота родителя не увеличивается. Это не хорошо. Никогда не вкладывайте внутрь строчных элементов блочные. В данном случае блочный тег H2 обернут в строчный span. Если уж возникла такая необходимость, пишите:

span {
	display: block;
}

 

Link to comment
Share on other sites

1 час назад, IvanRu08 сказал:

Сразу бросается в глаза

Так это фон :) Его лучше сделать паттерном в фотошопе и зарепитить по двум осям. Дизайн древний, поэтому верстка должна соответствовать тем временам. Да и лучше брать макеты трендовые, т.к. в 2016 такими шаблонами тяжело кого-то заинтересовать и тем более удивить.

Link to comment
Share on other sites

1 минуту назад, by chris сказал:

Так это фон :) Его лучше сделать паттерном в фотошопе и зарепитить по двум осям. Дизайн древний, поэтому верстка должна соответствовать тем временам. Да и лучше брать макеты трендовые, т.к. в 2016 такими шаблонами тяжело кого-то заинтересовать и тем более удивить.

Может быть мне везет, но всегда попадались макеты, где можно посмотреть ширину высоту паттерна, который используется и без проблем через repeat без стыков все соединить верстке. Хотя, макет древний, согласен:) 

Link to comment
Share on other sites

Смотрится все неплохо, но большинство блоков имеют абсолютное позиционирование, без которого вполне можно было обойтись, и фиксированную высоту.

И вот что происходит, если текста в одном из блоков стало больше:

http://prntscr.com/brdsql

Font-family, font-size, line-height можно одной строкой написать например, ну пожалуй впервые вижу чтобы размер шрифта в пунктах задавали 

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 Mondeus
      Доброго всем времени суток. Прошу помощи. Научите принципу изменения футера. Движок 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 } }  

    • By zeiger2
      Здравствуйте! У меня стоит задача, что при наведении на блок li строка должна поменять цвет, в том числе и картинка. Я меняю картинку с помощью 
      background-image: none;     background: url(../img/check_icon_red.png) left no-repeat;   Но теперь картинка позицианируется не там где должна, её можно поставить на место только вручную, через -100px. Нужно поставить ровно туда, где она была. Должна быть в одном ряду с другими
    • By Mix9
      есть див с 5 img, при уменьшении экрана див выходит за него. Я добавил overflow: auto для этого div в надежде на то, что я смогу прокручивать фотки с помощью скроллбара, однако даже с ним почему-то я не вижу часть фоток которые вышли за границу. Что с этим можно сделать? класс video повторяется 5 раз, я тут оставил только 1 
      .content{ width: 90%; background-color: #333; } .video{ margin: 0px 4px 0px 4px; width: 310; display: flex; flex-direction:column; } .video_button_text{ margin-top: 10px; display: flex; flex-direction: row; font-size: 20px; color: white; } .video_text_div{ display: inline-block; width: 250px; } .video_text{ text-align: justify-all; margin: 0px; display: -webkit-box; -webkit-line-clamp: 2; -webkit-box-orient: vertical; overflow: hidden; } .slidan_videos{ margin: 0px 10px 0px 20px; overflow: auto; width: auto; margin-bottom: 50px; display: flex; flex-direction: row; justify-content: space-around; } <div class="content"> <div class = slidan_videos> <div class = video> <div> <a href = 'ссылка'><img class="img" src=""картинка"></a> </div> <div class = video_button_text> <div class = avatarka_div> <a href="ссылка" target="_blank"><img class = avatarka src="картинка"></a> </div> <div class = video_text_div> <p class = video_text><a href="ссылка">текст</a></p> </div> </div> </div>
    • By Марко
      Добрый день. Начинающий программист, столкнулся с проблемой. Селектор .class не работает должным образом. Несмотря на правильное, я надеюсь, описание, на web-странице не отображается ни одно изображение. С чем может быть связано? Заранее спасибо за помощь. 



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