Jump to content

Хотелось бы услышать комментарии по верстке.


MrJeffry
 Share

Recommended Posts

Всем привет! Отдаю на ваш суд свою первую верстку, косячную до безобразия, как мне кажется. Но тем не менее очень хотелось бы услышать комментарии профессионалов. Заранее спасибо за то что потратите свое время!!!

 

Моя верстка

Link to comment
Share on other sites

За чем вам 2 тега h1 да еще и подряд да еще и второй тег с телефоном?

поисковики не оценят

 

Какой смысл обрамлять 2 тега h1 в тег article?  Про тег div забыли и зря не используете

 

Нашел еще 2 тега h1 итого уже 4.  Поисковики негодуют. Пользователям естественно все равно хоть h0 поставить ))

H2 тоже не к месту.

 

 

А в CSS

Чего у вас у каждого элемента отрицательные маргины? :o

дальше смотреть не стал

Link to comment
Share on other sites

Для начала вам следовало бы проверить свою вёрстку на валидацию, не правильно используете теги article с точки зрения семантики. Ловите 3 варнинга. Используйте div заместо них раз нету заголовков h2-h6 и текст не несёт какой-то осмысленной части. А вообще почитайте про теги div, article, section http://noteskeeper.ru/54/ где и как надо их использовать, так как по вёрстке с section тоже местами перемудрили. Ну и конечно, как писали выше, используете h1 больше 1 раза

Warning: Article lacks heading. Consider using h2-h6 elements to add identifying headings to all articles.From line 29, column 5; to line 29, column 27ader>↩    <article class="order">↩     Warning: Article lacks heading. Consider using h2-h6 elements to add identifying headings to all articles.From line 76, column 5; to line 76, column 27icle>↩    <article class="info2">↩     Warning: Article lacks heading. Consider using h2-h6 elements to add identifying headings to all articles.From line 94, column 5; to line 94, column 27icle>↩    <article class="order">↩     
Link to comment
Share on other sites

1) Нет title;

2) Ховер в меню не надо таким делать, погуглите чем отличается сплошное подчёркивание у ссылок от прерывистого;

3) Двигаю ползунки на слайдере и ничего не меняется;

4) Чуть-чуть съехала сетка у списков слева вверху, заголовок чуть правее текста;

5) "Вы" с заглавной буквы не в начале предложения не по правилам русского языка пишут задницеоблизыванты;

6) "Вы оставляете заяку" по слову на строчку плохо выглядит;

7) У формы нет проверки;

8) Тексты типографить надо, даже рыбу;

9) Вёрстка внутри конечно тот ещё не подарок, но от разнесения полей ввода на одну форму, а кнопки отправки на вторую форму, я прибалдел немало.

Link to comment
Share on other sites

 

Для начала вам следовало бы проверить свою вёрстку на валидацию, не правильно используете теги article с точки зрения семантики. Ловите 3 варнинга. Используйте div заместо них раз нету заголовков h2-h6 и текст не несёт какой-то осмысленной части. А вообще почитайте про теги div, article, section http://noteskeeper.ru/54/ где и как надо их использовать, так как по вёрстке с section тоже местами перемудрили. Ну и конечно, как писали выше, используете h1 больше 1 раза

Warning: Article lacks heading. Consider using h2-h6 elements to add identifying headings to all articles.From line 29, column 5; to line 29, column 27ader>↩    <article class="order">↩     Warning: Article lacks heading. Consider using h2-h6 elements to add identifying headings to all articles.From line 76, column 5; to line 76, column 27icle>↩    <article class="info2">↩     Warning: Article lacks heading. Consider using h2-h6 elements to add identifying headings to all articles.From line 94, column 5; to line 94, column 27icle>↩    <article class="order">↩     

Я делал валидацию, и предупреждения эти тоже видел, просто не предал им особого значения) Спасибо за ссылку, изучу этот вопрос детальнее.

 

1) Нет title;

2) Ховер в меню не надо таким делать, погуглите чем отличается сплошное подчёркивание у ссылок от прерывистого;

3) Двигаю ползунки на слайдере и ничего не меняется;

4) Чуть-чуть съехала сетка у списков слева вверху, заголовок чуть правее текста;

5) "Вы" с заглавной буквы не в начале предложения не по правилам русского языка пишут задницеоблизыванты;

6) "Вы оставляете заяку" по слову на строчку плохо выглядит;

7) У формы нет проверки;

8) Тексты типографить надо, даже рыбу;

9) Вёрстка внутри конечно тот ещё не подарок, но от разнесения полей ввода на одну форму, а кнопки отправки на вторую форму, я прибалдел немало.

 

Как развернуто, большое спасибо, но у меня есть несколько вопросов касательно Вашего ответа:

8) Как понять пример с рыбой?

9) А почему так делать нельзя? Верстка да...я только вчера узнал почему у меня отступы вложенных элементов добавляли отступы родительскому элементы, так называемое "выпадание" внешних отступов.Но будем работать! Спасибо большое!

Link to comment
Share on other sites

А почему так делать нельзя?

Потому что у вас одна единая форма с полями для ввода и кнопкой отправки. А так получается, что у вас кнопка отправки отвалилась.

 

Как понять пример с рыбой?

https://www.google.ru/search?client=opera&q=текст+рыба&sourceid=opera&ie=UTF-8&oe=UTF-8 

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