Jump to content

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


SvDen
 Share

Recommended Posts

Добрый день, уважаемые участники форума.

 

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

 

Два нюанса:

а) ошибка с названиями классов camelCase мне известна;

б) слайдеры - это велосипеды на чистом JS и да, их можно было бы сделать резиновыми, но было лень ) всё равно в реальном проекте проще на jquery писать. Или я здесь не прав?

 

Собственно сам сайт: http://svden.github.io/PracticeEcommerce/

Макет (переведен в JPG): https://gyazo.com/4dcba975a038e1f77e87a6d053b0f34c

 

Что улучшить? Где нужно было делать иначе? В общем, буду рад любым замечаниям (особенно развёрнутым ;).

 

Спасибо.

 

Link to comment
Share on other sites

Здравствуйте, сразу скажу, я не профи, но кое-что подсказать могу.

На разрешении 450px по ширине:
При раскрытии меню, contact us прилипает к блоку с товарами. Надо чтобы не прилипал, может быть паддинг добавить.

 

800px по ширине:
Предлагаю сделать пункты меню по центру и заголовок "Featured products" тоже поставить по центру. Попробуйте так сделать. Быть может я ошибаюсь, но по-моему так смотрится лучше.

Link to comment
Share on other sites

Да, верно, по центру было бы лучше. Спасибо.

Насчёт "При раскрытии меню, contact us прилипает к блоку с товарами." немного не понял. У меня вид вот такой: https://gyazo.com/43a1245080ab4e49756eb42a60a2f970

Link to comment
Share on other sites

Зайдите сюда: http://quirktools.com/screenfly/#u=http%3A//svden.github.io/PracticeEcommerce/&w=240&h=320&a=31&s=1

Нажмите на ваше MENU, прокрутите до пункта CONTACT US. У меня последний пункт прилипает вплотную к верхней границе блока с белым фоном. Почему у вас по-другому отображает не знаю. Проверила на screenfly во всех мобильниках, зашла на http://ami.responsivedesign.is/везде одно и тоже, может у вас сервис (которым вы пользуетесь) не правильно отображает адаптивность?

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 Seadjus
      Добрый день!
      Огромная просьба камнями не кидаться!
      Начал изучать JS , пока нет особых продвижений в изучении.
      помогите оптимизировать игру в пары. по условиям задания необходимо реализовать игру в пары с таймером, выбором количества карт .
      Ниже
      https://codepen.io/Seadjus/pen/VwVedaP
    • 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 ilya1203
      Прошу помочь с заданием:
      Написать файл скрипта, который генерирует три числа, соответствующие значениям высоты, длины и ширины комнаты. Отобразить в консоли:
      a.Площадь стен комнаты
      b.Объём комнаты
      c.*Объём комнаты за вычетом случайно сгенерированного количества чисел, которые определяет объёмы случайно зашедших в комнату людей.
      (в итоге запустит скрипт последством консоли)
      (заранее огромное спасибо)
×
×
  • 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