Jump to content

Верстка очередного макета


Pashkes
 Share

Recommended Posts

Оцените пожалуйста верстку простенького лэндинга, фикс, использован препроцессор Sass,  флексы,  именование классов выполнено по БЭМ

ссылка на работу - https://pashkes.github.io/

ссылка на репозиторий - https://github.com/pashkes/pashkes.github.io

ссылка на макет - https://drive.google.com/open?id=0B2WaTCbThw1GR0dmb3d0V2J2S28

 

Edited by Pashkes
Link to comment
Share on other sites

Только что, Endorphin сказал:

Надо бы адаптивку провести, на iphone5 все блоки которые находились справа выпали за пределы основного контейнера

так адаптив и не задумывался, тем более макет только в десктоп версии. А, все, понял косяк, исправлю

Edited by Pashkes
Link to comment
Share on other sites

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

Надо бы адаптивку провести, на iphone5 все блоки которые находились справа выпали за пределы основного контейнера

Забыл для body добавить min-width: 1200px

Link to comment
Share on other sites

- У атрибута href для ссылки одного лишь "tel:" недостаточно. Номер тоже необходим.
- У кнопки поиска так же, как и у социальных кнопок, узел с текстом должен его содержать.
- Контейнер с паддингами по 15 пикселей по бокам в связке с content-box сделает отступы более явными. Сейчас их видимость зависит от минимальной ширины тела страницы, что не организуешь на адаптивном макете. Поэтому, отступы стоит учитывать сразу.
БЭМ
- Элемент header__translucent-bg не помешает изменить на container, а header__border-bottom именовать, как header__row... Т.е примерно такая разметка, на мой взгляд рациональнее -

<header class="header">
  <div class="container">
    <div class="header__row header__row--border-bottom"></div>
    <div class="header__row"></div>
  </div>
</header>

- Лого, чаще всего независимый блок, я делаю его отдельно. Ведь, часто лого встречается и в футере. Каким боком туда попал  логотип джумлы?)

- Блок contacts размечен по-бэмски, а вот блок social, что-то безумное. Плюс от этого класса - search-tour__field-wrapper search-tour__field-wrapper--icon-search, голова пошла кругом. Есть объяснения?)

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

- В целом сверстано на славу. Только если использовать абстракцию по БЭМу, то немножко усерднее. Чтобы добиться модульности для копирования в наступные проекты).  

  • Like 2
Link to comment
Share on other sites

3 часа назад, by chris сказал:

- Элемент header__translucent-bg не помешает изменить на container, а header__border-bottom именовать, как header__row... Т.е примерно такая разметка, на мой взгляд рациональнее -


<header class="header">
  <div class="container">
    <div class="header__row header__row--border-bottom"></div>
    <div class="header__row"></div>
  </div>
</header>

 

header__translucent-bg это тянущийся полупрозрачный темный слой, зачем его изменять на container если он фиксированной ширины 

3 часа назад, by chris сказал:

- Блок contacts размечен по-бэмски, а вот блок social, что-то безумное. Плюс от этого класса - search-tour__field-wrapper search-tour__field-wrapper--icon-search, голова пошла кругом. Есть объяснения?)

А что с блоком social не так, обычный список со ссылками внутри. 

search-tour__field-wrapper  search-tour__field-wrapper--icon-search это обертка поля ввода для иконок

<div class="social">
	<ul class="social__list">
		<li><a class="social__item  social__item--fb" href="#">fb</a></li>
		<li><a class="social__item  social__item--tw" href="#">tw</a></li>
		<li><a class="social__item  social__item--insta" href="#">insta</a></li>
		<li><a class="social__item  social__item--youtube" href="#">youtube</a></li>
		<li><a class="social__item  social__item--viber" href="#">viber</a></li>
	</ul>
</div>

Так если нету больше других версий макета (мобильной, планшетной), мне что нужно было самому придумать как должны себя вести блоки на других девайсах?

Спасибо за уделенное время на меня:)

На днях выложу верстку первого адаптивного проекта который делал на продвинутом интенсиве htmlAcademy, 3 страницы, каждая страница в 3 версиях. Мне конечно уже его проверяли, но мне кажется не достаточно тщательно, хоть я и сдал его на 97%

Edited by Pashkes
Link to comment
Share on other sites

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

header__translucent-bg это тянущийся полупрозрачный темный слой, зачем его изменять на container если он фиксированной ширины 

Да, немножко поспешил с контейнером. Конечно же из-за бордера под телефонами такая разметка не сойдет. 
1. У header__top при скролле можно изменять стейт яваскриптом удаляя класс модификатор при прокрутке. А по умолчанию задать (header__top) цвета инвертированные с transition. Или наоборот. Актуально для фиксированного блока.

2. Модификатор с цветом бордера, по умолчанию возможен другой, или его отсутствие. Возможны еще какие-то кастомные стили.
3. Для header__bottom по умолчанию паддинги, а header__bottom--blue, лишь модификатор с фоном и цветом текста. 

Шапка сложнейший элемент на сайте, и его правильное структурирование поможет быстро развертывать похожую разметку на любом проекте, по подготовленным патернам.

<header class="header">
  <div class="header__top header__top--translucent-bg"> // 1
    <div class="header__row header__row--border-bottom"> // 2
      <div class="container"></div>
    </div>
    <div class="header__row">
      <div class="container"></div>
    </div>
  </div>

  <div class="header__middle">
    <div class="container"></div>
  </div>

  <div class="header__bottom header__bottom--blue"> // 3
    <div class="container"></div>
  </div>
</header>

 

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

А что с блоком social не так, обычный список со ссылками внутри. 

Действительно, все хорошо. После фильма настроил режим, который засветил маленький пробел и я его не заметил. Показалось все в одну строку, по сравнению с классами contacts__list контраст сильнейший. И сейчас открываю, действительно не видно пробелы).
Да, это хорошо, что ты так критично подошел к моему посту. Из тебя получиться, отличный разработчик.

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

Так если нету больше других версий макета (мобильной, планшетной), мне что нужно было самому придумать как должны себя вести блоки на других девайсах?

Скажу по секрету. Я так и работаю. В моей компании, дают только дизайн под десктоп, а дальше сам все додумываю.

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

Спасибо за уделенное время на меня:)

Мне интересно просматривать чужие работы, на свои уже устал созерцать.)

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

На днях выложу верстку первого адаптивного проекта который делал на продвинутом интенсиве htmlAcademy, 3 страницы, каждая страница в 3 версиях. Мне конечно уже его проверяли, но мне кажется не достаточно тщательно, хоть я и сдал его на 97%

Отлично, буду ждать. В предыдущем посте я ошибся 2 раза. Все ошибаются. 

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 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-странице не отображается ни одно изображение. С чем может быть связано? Заранее спасибо за помощь. 



    • By Kaido
      Использую готовый плагин для модальных окон(от MaxGraph). Проблема в том, что когда у меня открыто два модальных окна, для примера Форма + Политика конфендициальности, и мне нужно закрыть политику вместе с ней закрывается и другое модальное окно. В JS я не сильно разбираюсь(собственно из за этого и использую готовый плагин), можете помочь кто работал с этим плагином? Я примерно понимаю как он работает, но реализовать чтобы закрывалось только одно не получается.
       
        <div class="content"> <button class="modal-btn" data-path="first" data-animation="fadeInUp" data-speed="1500">Открыть окно 1</button> </div> <div class="modal"> <div class="modal__wrapp" data-target="first"> <div class="modal__content"> <button class="modal__close">Закрыть</button> модальное окно <button data-path="policy">Политика</button> </div> </div> <div class="modal__wrapp" data-target="policy"> <div class="modal__content"> <button class="modal__close">Закрыть</button> политика </div> </div> </div> .modal { --transition-time: 0.3s; position: fixed; left: 0; top: 0; right: 0; bottom: 0; z-index: 1000; cursor: pointer; overflow-y: auto; overflow-x: hidden; text-align: center; opacity: 0; visibility: hidden; transition: opacity var(--transition-time), visibility var(--transition-time); } .modal__wrapp { display: none; cursor: default; width: fit-content; height: fit-content; } .modal__content{ position: absolute; left: 500px; width: 500px; height: 500px; display: flex; color: white; flex-direction: column; text-align: left; background-color: #000; } .modal__content button{ width: 200px; height: 50px; margin: 50px 0; } .modal.is-open { opacity: 1; visibility: visible; transition: opacity var(--transition-time), visibility var(--transition-time); } .modal__wrapp.modal-open { display: flex; } .disable-scroll { position: relative; overflow: hidden; height: 100vh; position: fixed; left: 0; top: 0; width: 100%; } .fade { opacity: 0; transition: opacity var(--transition-time); } .fade.animate-open { opacity: 1; transition: opacity var(--transition-time); } .fadeInUp { opacity: 0; transform: translateY(vw(-100)); transition: opacity var(--transition-time), transform var(--transition-time); } .fadeInUp.animate-open { opacity: 1; transform: translateY(0); transition: opacity var(--transition-time), transform var(--transition-time); } .modal__wrapp[data-target="policy"] .modal__content{ left: 1050px; background-color: #000; opacity: .5; } class Modal { constructor(options) { let defaultOptions = { isOpen: () => {}, isClose: () => {}, } this.options = Object.assign(defaultOptions, options); this.modal = document.querySelector('.modal'); this.speed = false; this.animation = false; this.isOpen = false; this.modalContainer = false; this.previousActiveElement = false; this.fixBlocks = document.querySelectorAll('.fix-block'); this.focusElements = [ 'a[href]', 'input', 'button', 'select', 'textarea', '[tabindex]' ]; this.events(); } events() { if (this.modal) { document.addEventListener('click', function(e){ const clickedElement = e.target.closest('[data-path]'); if (clickedElement) { let target = clickedElement.dataset.path; let animation = clickedElement.dataset.animation; if (clickedElement.classList.contains('modal-close')) { this.close(); } let speed = clickedElement.dataset.speed; this.animation = animation ? animation : 'fade'; this.speed = speed ? parseInt(speed) : 300; this.modalContainer = document.querySelector(`[data-target="${target}"]`); this.open(); return; } if (e.target.closest('.modal__close')) { this.close(); return; } }.bind(this)); window.addEventListener('keydown', function(e) { if (e.keyCode == 27) { if (this.isOpen) { this.close(); } } if (e.keyCode == 9 && this.isOpen) { this.focusCatch(e); return; } }.bind(this)); this.modal.addEventListener('click', function(e) { if (!e.target.classList.contains('modal__wrapp') && !e.target.closest('.modal__wrapp') && this.isOpen) { this.close(); } }.bind(this)); } } open() { this.previousActiveElement = document.activeElement; this.modal.style.setProperty('--transition-time', `${this.speed / 1000}s`); this.modal.classList.add('is-open'); this.disableScroll(); this.modalContainer.classList.add('modal-open'); this.modalContainer.classList.add(this.animation); setTimeout(() => { this.options.isOpen(this); this.modalContainer.classList.add('animate-open'); this.isOpen = true; this.focusTrap(); }, this.speed); } close() { if (this.modalContainer) { this.modalContainer.classList.remove('animate-open'); this.modalContainer.classList.remove(this.animation); this.modal.classList.remove('is-open'); this.modalContainer.classList.remove('modal-open'); this.enableScroll(); this.options.isClose(this); this.isOpen = false; this.focusTrap(); } } focusCatch(e) { const focusable = this.modalContainer.querySelectorAll(this.focusElements); const focusArray = Array.prototype.slice.call(focusable); const focusedIndex = focusArray.indexOf(document.activeElement); if (e.shiftKey && focusedIndex === 0) { focusArray[focusArray.length - 1].focus(); e.preventDefault(); } if (!e.shiftKey && focusedIndex === focusArray.length - 1) { focusArray[0].focus(); e.preventDefault(); } } focusTrap() { const focusable = this.modalContainer.querySelectorAll(this.focusElements); if (this.isOpen) { focusable[0].focus(); } else { this.previousActiveElement.focus(); } } disableScroll() { let pagePosition = window.scrollY; this.lockPadding(); document.body.classList.add('disable-scroll'); document.body.dataset.position = pagePosition; document.body.style.top = -pagePosition + 'px'; } enableScroll() { let pagePosition = parseInt(document.body.dataset.position, 10); this.unlockPadding(); document.body.style.top = 'auto'; document.body.classList.remove('disable-scroll'); window.scroll({ top: pagePosition, left: 0 }); document.body.removeAttribute('data-position'); } lockPadding() { let paddingOffset = window.innerWidth - document.body.offsetWidth + 'px'; this.fixBlocks.forEach((el) => { el.style.paddingRight = paddingOffset; }); document.body.style.paddingRight = paddingOffset; } unlockPadding() { this.fixBlocks.forEach((el) => { el.style.paddingRight = '0px'; }); document.body.style.paddingRight = '0px'; } } const modal = new Modal({ isOpen: (modal) => { console.log(modal); console.log('opened'); }, isClose: () => { console.log('closed'); }, });  
×
×
  • 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