Jump to content

Оцените верстку


Maryska
 Share

Recommended Posts

Здравствуйте! Оцените, пожалуйста, верстку.

Сайт http://probjob.esy.es

Макет 3 страниц http://vladmaxi.net/web-designer/makety-sajtov/psd-maket-dlya-biznes-sajta-bislite-dizajn-2-x-stranic.html

Ie 9+

Еще хочу сразу задать вопрос на странице  CONTACT US в footer inline-block не хотят растягиваться по ширине в Opera и Сhrome, как я поняла это связано как то с формой расположенной выше, но как это исправить не знаю, подскажите кто знает

Edited by Maryska
  • Like 1
Link to comment
Share on other sites

Непонятно, зачем было делать попытку резины, если сам макет этого не предполагал изначально.

Причем из-за этой попытки некоторые части выглядят криво.

Edited by advokatua
Link to comment
Share on other sites

Непонятно, зачем было делать попытку резины, если сам макет этого не предполагал изначально.

Причем из-за этой попытки некоторые части выглядят криво.

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

Link to comment
Share on other sites

как по мне резиновая верстка это уже не так актуально, сейчас эра responsive-лично моё мнение.


center contener—-весь блок хорошо сделан у новичков не часто такое встретишь(с точки зрения html), только вот еще поработать с названиями классов(прийдет с опытом),  contener-не так пишется ;)

 

по поводу заголовка не нужно  в h1 вот сегодня писал об этом:

 

Значит так, лично заметки от меня ( эксперты, если где не прав поправят меня)

1) Зачем для логотипа дополнительный div? Разве нельзя обойтись просто тегом h1?
Удачи

А Зачем логотипу h1?  а так как логотип скорее всего будет повторяться по всему сайту, то и заголовок будет одинаков везде это не правильно! и после того как этот сайт натянут на cms, то когда юзер будет кликать на новость, его будет беребрасывать на страницу одного поста, где h1 будет как раз заголовок новости, причем тут логотип??

Лучше обернуть для более конструированного html

 

<div class="logo"><img src="......"</div>

по поводy header, footer-лучше добавлять классы.

 

а так приличная  работа(то что просмотрел)

 

просмотрел только первую стран. но не оч. внимательно))

Edited by cris7iano
Link to comment
Share on other sites

как по мне резиновая верстка это уже не так актуально, сейчас эра responsive-лично моё мнение.

center contener—-весь блок хорошо сделан у новичков не часто такое встретишь(с точки зрения html), только вот еще поработать с названиями классов(прийдет с опытом),  contener-не так пишется ;)

 

по поводу заголовка не нужно  в h1 вот сегодня писал об этом:

 

Значит так, лично заметки от меня ( эксперты, если где не прав поправят меня)

1) Зачем для логотипа дополнительный div? Разве нельзя обойтись просто тегом h1?

Удачи

А Зачем логотипу h1?  а так как логотип скорее всего будет повторяться по всему сайту, то и заголовок будет одинаков везде это не правильно! и после того как этот сайт натянут на cms, то когда юзер будет кликать на новость, его будет беребрасывать на страницу одного поста, где h1 будет как раз заголовок новости, причем тут логотип??

Лучше обернуть для более конструированного html

 

<div class="logo"><img src="......"</div>

по поводy header, footer-лучше добавлять классы.

 

а так приличная  работа(то что просмотрел)

 

просмотрел только первую стран. но не оч. внимательно))

Спасибо! По поводу классов к footer и header, вы имейте виду <footer class="footer">…</footer>, а зачем так делать, разве в HTML5 ввели эти теги не для того чтоб ими пользоваться без классов, или я чего то не поняла? 

Link to comment
Share on other sites

Спасибо! По поводу классов к footer и header, вы имейте виду …

, а зачем так делать, разве в HTML5 ввели эти теги не для того чтоб ими пользоваться без классов, или я чего то не поняла?

 

 

затем что в HTML5 на странице может быт множество <header> <footer> в рамках того же <section>

  • Like 1
Link to comment
Share on other sites

 

Спасибо! По поводу классов к footer и header, вы имейте виду …

, а зачем так делать, разве в HTML5 ввели эти теги не для того чтоб ими пользоваться без классов, или я чего то не поняла?

 

 

затем что в HTML5 на странице может быт множество <header> <footer> в рамках того же <section>

 

Я знаю, что header footer может использовать не один раз, но в данном случае  только один. Если конечно учитывать что может быть дописано еще что- то тогда да. Спасибо, учту в будущем. 

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