Skip to content

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

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

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

Conversation

@ilearnf

@ilearnf ilearnf commented Mar 29, 2018

Copy link
Copy Markdown

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