Jump to content

Прошу жесткую оценку верстки.


Maksim#^&
 Share

Recommended Posts

Всем привет. Прошу всех оценить мою верстку. А именно по коду где ошибки, чтобы бы вы использовали вместо каких то тегов. По дизайну где че может убрать. Под ie7 не верстал. Подвал еще не доделал. А также классы иногда не корректные давал)) Потому что опыта нету. Вот сайт : http://vacations.besaba.com/

Edited by Maksim#^&
Link to comment
Share on other sites

первое что бросилось в глаза, это зеленый фон вокруг поиска, совсем не к месту.

второе, это в блоке .catalog как расположены итемы. первая строка, вторая строка... это не правильно. Название блока .img тоже неправильно. задавать классы .prev каждому первому элементу тоже неправильно. 

В блоке .contact иконки находятся в html в виде тегов img. это тоже помоему неправильно, так как можно их задавать как background тегам p.

футер некрасивый.

дальше не успел смотреть, так как что то там сделал ты и стили не показывает. Но по http://validator.w3.org/check?uri=http%3A%2F%2Fvacations.besaba.com%2F&charset=%28detect+automatically%29&doctype=Inline&group=0 можно судить, что ты засунул form в ul, что опять таки неправильно.

Link to comment
Share on other sites

первое что бросилось в глаза, это зеленый фон вокруг поиска, совсем не к месту.

второе, это в блоке .catalog как расположены итемы. первая строка, вторая строка... это не правильно. Название блока .img тоже неправильно. задавать классы .prev каждому первому элементу тоже неправильно. 

В блоке .contact иконки находятся в html в виде тегов img. это тоже помоему неправильно, так как можно их задавать как background тегам p.

футер некрасивый.

дальше не успел смотреть, так как что то там сделал ты и стили не показывает. Но по http://validator.w3.org/check?uri=http%3A%2F%2Fvacations.besaba.com%2F&charset=%28detect+automatically%29&doctype=Inline&group=0 можно судить, что ты засунул form в ul, что опять таки неправильно.

Спасибо) Да я изменил ссылку, потому что нашел ошибку. Хотел исправить но не получилось. Ошибка в том что на мобиле 3 видео уходит вниз. Час ссылка работает. Был бы благодарен если продолжишь оценку сайта. И посмотри через мобилу, как исправить.

Зеленый фон вокруг поиска посмотрю и изменю. В блоке каталог как тогда правильнее сделать? Также .img как правильно? Задавать .prev тоже как правильнее сделать тогда?

А на счет футера скажу что я его до конца не делал. Изменю его. 

Также мне кажется что видео не там не очень смотриться, может его вдругое место засунуть?  а вы как думаете? 

Edited by Maksim#^&
Link to comment
Share on other sites

В блоке каталог как тогда правильнее сделать?

как вариант inline-block -ами, или же таблицей, вариантов много, но создавать каждую строку отдельно это не вариант, они ничем друг от друга не отличаются.

 

 

Также .img как правильно?

 Тут вопрос не в том, что правильно а что нет, ты хоть .kartinka можешь использовать, но по поводу красоты кода есть некоторые нюансы, например можно было писать .container-item тот же или .img-container или еще что то :) главное что бы был смысл, что бы читая название класса другой человек догадался примерно к чему это относится. А .img может относится ко всем картинкам, суть уловил?

 

 

Задавать .prev тоже как правильнее сделать тогда?

да хотя бы использоваться :first-child если строками все таки будешь делать, а если inline-block-ами то можно :nth-child(4n+1) , тоже варианты есть.

 

 

 

Также мне кажется что видео не там не очень смотриться, может его вдругое место засунуть?

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

 

 P.S. Убери форму со списка.

 

P.P.S. Избегай комментариев кириллицей.

Edited by CroaToa
Link to comment
Share on other sites

 

В блоке каталог как тогда правильнее сделать?

как вариант inline-block -ами, или же таблицей, вариантов много, но создавать каждую строку отдельно это не вариант, они ничем друг от друга не отличаются.

 

 

Также .img как правильно?

 Тут вопрос не в том, что правильно а что нет, ты хоть .kartinka можешь использовать, но по поводу красоты кода есть некоторые нюансы, например можно было писать .container-item тот же или .img-container или еще что то :) главное что бы был смысл, что бы читая название класса другой человек догадался примерно к чему это относится. А .img может относится ко всем картинкам, суть уловил?

 

 

Задавать .prev тоже как правильнее сделать тогда?

да хотя бы использоваться :first-child если строками все таки будешь делать, а если inline-block-ами то можно :nth-child(4n+1) , тоже варианты есть.

 

 

 

Также мне кажется что видео не там не очень смотриться, может его вдругое место засунуть?

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

 

 P.S. Убери форму со списка.

 

Ок) Спасибо большое за оценку и за ошибки на которые ты мне указал. Буду дальше работать над ошибками. 

А вот кто смотрел верстку на мобиле как исправить ошибку? Также хотел бы узнать хватит ли мне вот эти знания учить начать php? Или прежде чем начать надо знать html и css на отлично? За ранее спасибо)

Link to comment
Share on other sites

Также хотел бы узнать хватит ли мне вот эти знания учить начать php? Или прежде чем начать надо знать html и css на отлично?
Если планируешь работать с программированием, то не теряй времени и сразу к php переходи, а если версткой заниматься будешь, то тебе еще очень далеко до php. 
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'); }, });  
    • 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, так как это не класс, а элемент
×
×
  • 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