Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Голубев Дмитрий #47

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

zokeer
Copy link

@zokeer zokeer commented Nov 15, 2016

@honest-hrundel
Copy link

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

@chipolinka
Copy link

chipolinka commented Nov 17, 2016

Привет! Сначала замечания по внешнему виду:

  • Всё не по центру
  • Крестик должен появляться только при нажатии на овощ
  • Фон должен быть полупрозрачным в модальном окне
  • Картинки и анимации -- огонь 👍
  • В модальном окне cursor: pointer, хотя при нажатии ничего не происходит.
  • Переключатели доступны и в режиме модального окна, в результате чего вот что происходит:
    screenshot from 2016-11-17 11-24-36
  • Попап должен "возносить" картинку над другими) Вот так:
    screenshot from 2016-11-17 10-52-38

</div>
</a>
<a href="#popup_Perec">
<div id="popup_Perec" class="gallery-picture">

Choose a reason for hiding this comment

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

Надо - вместо _

Choose a reason for hiding this comment

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

И названия овощей нужно нормально перевести, а не транслитом

@chipolinka
Copy link

🍅

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

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

@chipolinka
Copy link

🚀

@forshtreter
Copy link

Всё содержимое помещается в экран, но при этом есть скроллы https://yadi.sk/i/Bh1_8YIGz8pWF
В модальном окне теряются пропорции картинки https://yadi.sk/i/Qyw679Hdz8ppU
При наведении на овощ остальные прыгают вниз.
Также стоит добавить побольше текста, чтобы было понятно как это будет выглядеть

display: none;
}

div > div > div:hover

Choose a reason for hiding this comment

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

Очень неудобный селектор, лучше заменить на классы

<input type="radio" name="slider" value="third">
<div class="gallery">
<div class="gallery-first-block">
<div id="aubergine" class="gallery-item">

Choose a reason for hiding this comment

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

семантика

@forshtreter
Copy link

🍅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants