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

Саитов Владислав #48

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

Саитов Владислав #48

wants to merge 3 commits into from

Conversation

nmkname
Copy link

@nmkname nmkname commented Nov 7, 2016

@honest-hrundel
Copy link

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

@nmkname
Copy link
Author

nmkname commented Nov 7, 2016

Привет. Пока css файл выглядит бедно и я не знаю, что туда добавить можно. Шрифты может или отступы какие. Как выровнять по нижней линии фотографии я не додумался, так что выглядит это неприятно. И я немного не понял про шкалу доброты(оставить эти звездочки или заменить на что-то?).

@honest-hrundel
Copy link

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

@m1ralx
Copy link

m1ralx commented Nov 8, 2016

image
image
image
image
выглядит пока не очень хорошо.. Эпилептикам лучше не водить быстро курсором по карточкам,

@m1ralx
Copy link

m1ralx commented Nov 8, 2016

🍅

@m1ralx
Copy link

m1ralx commented Nov 8, 2016

Звёздочки наверн можно оставить

@nmkname
Copy link
Author

nmkname commented Nov 8, 2016

Там был просто таск: сделать эффект какой-нибудь при наведении на фото. Ничего в голову лучше не пришло. Согласен, эпилептикам придется тяжко. Да и обычному человеку не особо приятно

@m1ralx
Copy link

m1ralx commented Nov 10, 2016

@nmkname нужно сделать хотя бы что-то нейтральное. Вряд ли ментор обрадуется такому

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.

3 participants