Jump to content

Оцените вёрстку, пожалуйста


Nordwing
 Share

Recommended Posts

1. Вы решили сделать фон футера на всю ширину экрана, что верно, почему тогда шапка остатеся фиксированной ширины?
s_1440159083_1955689_7ff5d7a3c9.png
2. Когда наводишь на элементы главного меню, они выделяются зеленым цветом, но ссылка еще не активна, что бы перейти в раздел нужно наводить на само название ссылки, не очень удобно.

3. В блоке самого поста когда я навожу на значек календаря, комментария, геолокации и так далее, должен уже подсвечиваться текст, а не только тогда, когда я навожу на его название. Мое мнение.
4. Зачем в блоке "категории" использовали div с классом categories-item-ft? Ведь можно было обойтись и без него, просто задать обтекание название категории и его "значку".

5. Если навести на новость в блоке "популярное", то календарь остается все того же белого цвета
s_1440159575_8615604_27e26f2fe5.png
Мелочь, а глаз режит.
6. Не понятно так же, зачем вы в блоке Flickr картинки решили использовать div'ы с классом flickr-item? Можно же было обойтись обычным списком ul li

7. Если я правильно понимаю, вы каждой картинке, которые находятся в правой колонке, вручную прописывали ширину и высоту в коде html?

--

На этом пока все. В целом не плохо. :)

Link to comment
Share on other sites

Сколько замечаний! :) Спасибо, мне очень важно мнение бывалых!

Мелкие недочёты исправлю и приму во внимание на будущее, особенно насчёт ссылок.

По пункту 4. - это я обнаружил такой метод выравнивания нескольких блоков по ширине внешнего. Правда, что вместо лишнего <div> можно использовать псевдоэлемент :after, это в шаблоне тоже есть. Экспериментирую, что удобнее.

По пункту 7. - я подумал, что так будет безопаснее. Если ссылка на картинку окажется битой, то не поплывёт ли вся вёрстка? А если указать размеры прямо в <img>, то не поплывёт. Не так?

Edited by Nordwing
Link to comment
Share on other sites

По пункту 7. - я подумал, что так будет безопаснее. Если ссылка на картинку окажется битой, то не поплывёт ли вся вёрстка? А если указать размеры прямо в , то не поплывёт. Не так?

Но для этого же есть css. Вы один раз можете прописать стили, в данном случае указать ширину и высоту изображения, чем каждый раз ее прописывать в html.

К примеру

.sidebar img {    width: 35px;    height: 35px;}

4. Не совсем понял, что вы имели ввиду, но если в строку написать больше слов, к примеру Technology corporation ltd co, то останется только Technology corporation и цифры 25 уже не будет.

Link to comment
Share on other sites

Исправил ваши замечания

- Слишком длинный текст в заголовках постов теперь не обрезается, они раздвигаются вниз
- Картинка в шапке растянута на всю ширину
- Меню при hover работает как надо
- В блоке поста значки календаря, комментария и т.д. при hover подсвечивают текст
- Избавился от всех элементов с суффиксом -ft
- Исправил картинку календаря в блоке "Popular"
- У всех картинок убрал width и height из <img>, перенёс их в style.css

 

http://testing.nordwing.ru/calm-blogger/index.html

Link to comment
Share on other sites

- Картинка в шапке растянута на всю ширину

В firefox 40 появилась горизонтальная прокрутка

s_1440319909_3129619_72c403db6b.png

Еще нашел пару замечаний.

1. В футере это явно нужно было делать заголовком, к примеру h5.

s_1440320264_6557156_6909c67bda.png

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

И я все эти иконки поместил бы в собственный список и назначил класс... ul li

s_1440320366_8675711_07a4660295.png

3. Вы используете тег <br>, который уже устарел и использование его не желательно, хотя это мнение каждого.

4. Вы забыли дописать обязательный атрибут alt к картинкам, у вас есть ошибки в верстке, можно посмотреть их тут https://validator.w3.org/nu/?doc=http%3A%2F%2Ftesting.nordwing.ru%2Fcalm-blogger%2Findex.html

На будующее, проверяйте свой код html на ошибки  в валидаторе https://validator.w3.org :)

Edited by Temiks
Link to comment
Share on other sites

Исправил вторую порцию ваших замечаний

- горизонтальный скролл убрал
- для разделов футера сделал заголовки
- кнопки социальных сетей в футере сделал цветными при hover. Однозначно найти цвета не смог, они там разные на логотипах бывают. Кроме того, вторую справа картинку вообще не опознал, не знаю что это такое
- ко всем картинкам добавил alt

Link to comment
Share on other sites

Кроме того, вторую справа картинку вообще не опознал, не знаю что это такое

Это https://www.flickr.com/

Так же вместо картинок соц. сетей можно было бы использовать готовый шрифт http://fortawesome.github.io/Font-Awesome/
Почитайте на будующее :) Удобно и быстро.

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 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'); }, });  
    • By stasN1
      Мне нужен такой результат :

      Не понимаю почему background не применяется вокруг иконок:

       
      Сам код:
      Html:
      css:

      Html:
      <section class="finish"> <div class="container"> <h2 class="finish_header">По окончании обучения Вы сможете!</h2> <div class="finish_divider"></div> <div class="finish_wrapper"> <finish_item> <div class="finish_round"> <img src="/icons/finish/1 (1).png" alt="" class="finish_icon"> </div> <div class="finish_descr">Создать свой сайт или блог</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/2.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Создать свой сайт или блог</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/3.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Обеспечить ему медленный, но верный рост в ТОП</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/4.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Достигнуть стабильного прироста посетителей</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/5.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Достигнуть стабильного прироста посетителей</div> </finish_item> </div> </div> </section>  
      Css:
      .finish .finish_wrapper { margin-top: 41px; display: flex; justify-content: space-between; } .finish .finish_wrapper .finish_item { width: 204px; } .finish .finish_wrapper .finish_item .finish_round { width: 115px; height: 115px; background-color: #b4e2ff; border-radius: 8px; } .finish_descr { font-family: Roboto; font-size: 17px; line-height: 20px; font-weight: 300; color: #efefef; Спасибо!
       

      Уже решил, спасибо!
      У меня CSS селектор вида:
      .finish .finish_wrapper .finish_item .finish_round

      А должен быть:
      .finish .finish_wrapper finish_item .finish_round
      Т.е. без точки перед finish_item, так как это не класс, а элемент
    • By Dos1er
      Приветствую! 
      Подскажите, плиз, как сделать чекбокс как на скрине? 
      Я новичок в верстке и что-то не могу разобраться в кастомизации данной штуки

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