Skip to content

Фомин Денис#18

Open
ilearnf wants to merge 2 commits intourfu-2017:masterfrom
ilearnf:master
Open

Фомин Денис#18
ilearnf wants to merge 2 commits intourfu-2017:masterfrom
ilearnf:master

Conversation

@ilearnf
Copy link
Copy Markdown

@ilearnf ilearnf commented Mar 29, 2018

No description provided.

@honest-hrundel honest-hrundel changed the title all Фомин Денис Mar 29, 2018
@honest-hrundel
Copy link
Copy Markdown

🍏 Пройден линтинг и базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍏 Пройден линтинг и базовые тесты

@VasiliiKuznecov
Copy link
Copy Markdown

image
наверное, неправильно, что так съезжают карточки влево?

@VasiliiKuznecov
Copy link
Copy Markdown

в макете принципиально, как выглядит меню и популярные товары, когда пунктов там много, добавь пожалуйста

@VasiliiKuznecov
Copy link
Copy Markdown

image

Список популярных сувениров смещен под блок с сувенирами
Форма заказа сувенира находится внизу страницы

У тебя эти блоки расположены неправильно

@VasiliiKuznecov
Copy link
Copy Markdown

Меню, список предлагаемых сувениров и список популярных сувениров растянуты на ширину экрана
Допускается наличие отступов от края экрана

У тебя на планшете ширина всегда фиксированная

@VasiliiKuznecov
Copy link
Copy Markdown

В шапке находится логотип сервиса и "бутерброд"

я еще понимаю Х - крестик, но заменять иконку бутерброда на W, кажется, перебор

@VasiliiKuznecov
Copy link
Copy Markdown

при скролле открывается меню, учитывая что нажатие на крестик еще почему-то откидывет наверх, добраться до формы и взаимодействовать с ней невозможно

@VasiliiKuznecov
Copy link
Copy Markdown

почему поле город типа number?

@VasiliiKuznecov
Copy link
Copy Markdown

При открытом меню содержимое страницы не скроллится

у тебя скроллится

@VasiliiKuznecov
Copy link
Copy Markdown

в режиме телефона если нажать на поиск и сбросить фокус с поля, поле пропадает

Comment thread index.js
document.getElementsByClassName('menu')[0].style.display = 'none';
});
document.getElementsByClassName('search__reveal')[0].addEventListener('click', function () {
document.getElementsByClassName('search__input')[0].style.display = 'block';
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

сделать через классы

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants