Jump to content

Оцените верстку и помогите решить проблему.

vintik

By vintik,
in Discussion of works

 Share

Recommended Posts

vintik Explorer

vintik

Posted
Posted

Здравствуйте. Прошу критики верстки и кода, буду очень благодарен за каждый полезный комментарий. И не понимаю в чем проблема, при ширине окна меньше 1200px, появляется отступ справа. 

https://vladislavmain.github.io/

Link to comment
Share on other sites
WebNewbe Explorer

WebNewbe

Posted
Posted
В 17.03.2018 в 23:00, vintik сказал:

Здравствуйте. Прошу критики верстки и кода, буду очень благодарен за каждый полезный комментарий. И не понимаю в чем проблема, при ширине окна меньше 1200px, появляется отступ справа. 

https://vladislavmain.github.io/

Почему в вашей навигации нет линков? Как переходить на другие страницы сайта? Добавьте в li тег a.
Далее, корзина и поиск. Делайте иконками, а не изображениями, они не кликабельны, при наведении никакй реакции. Надо править.
На кнопки добавьте hover эффекты.
В секции about, добавьте плавность на hover и не совсем понятно, они должны быть кликабельными или нет, если да, то добавьте внутрь линк a.
Вообще немного странная структура страницы. Класс футер должен быть в элементе footer, вместо div'ов на месте секций пропишите section. По поводу тега main, ознакомьтесь с ним по спецификации W3C.
Сам я использую такую структуру, может и вам по душе окажется 

<body>

	<!-- WRAPPER -->
	<div class="wrapper">

		<!-- HEADER -->
		<header class="header">
						
				
		</header>

		<!-- MAIN-CONTENT -->
		<main class="main-content">
			
		</main>
	</div>

	<!-- FOOTER -->
	<footer class="footer">
		
	</footer>

	<!-- scripts -->
	<script src="js/scripts.min.js"></script>

</body>

Едем дальше, аккордеоны в секции услуг, добавьте плавности им, и желательно при повторном нажатии на элемент его закрывать, в ином случае мы можем его закрыть, только нажав на другой блок.
У слайдеров стрелкам добавьте hover + cursor: pointer при наведении
В секции team, тоже самое, добавляем плавность при наведении. С соц линками у вас трабл, вы лучше не тегу li присвивайте высоту и ширину, а тегу <a> лучше, так как, чтобы перейти по этим линками приходится тыкать в самоу иконку, а это не удобно
Секцию с клиентами сделайте кликабельной и добавьте тут в линки target="_blank", чтобы при нажатии открывалась другая вкладка. Вашему юзеру не нужно лишний раз уходить с сайта:)
В секции работ, сделайте изображения ссылками, можно добавить сюда плагин magnific-popup
В общем тут везде одно, и в футере и в блогах и тд добавить ссылок, плавных hover-эффектов 
 

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
Go to topic listing

  • Similar Content

    • Mondeus
      Вёрстка футера на движке Xenforo
      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 } }  

    • zeiger2
      Смена background-image при работе с псевдоклассом hover
      By zeiger2
      Здравствуйте! У меня стоит задача, что при наведении на блок li строка должна поменять цвет, в том числе и картинка. Я меняю картинку с помощью 
      background-image: none;     background: url(../img/check_icon_red.png) left no-repeat;   Но теперь картинка позицианируется не там где должна, её можно поставить на место только вручную, через -100px. Нужно поставить ровно туда, где она была. Должна быть в одном ряду с другими
    • Mix9
      див уходит за границу экрана
      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>
    • Марко
      Не работает селектор .class
      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

  I accept