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

Маркевич Роман #59

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

Маркевич Роман #59

wants to merge 11 commits into from

Conversation

Tassadar2499
Copy link

@Tassadar2499 Tassadar2499 commented Nov 22, 2018

@honest-hrundel
Copy link

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

@dotokoto
Copy link

Всё разъехалось 🙈
2018-11-27 16-49-50
Со шрифтами и отступами всё сильно не так :( Попробуй подобрать более подходящий шрифт и отступы. И сделать кнопку "Найти" стрелочкой, как на макете. После этого посмотрю код

@dotokoto
Copy link

dotokoto commented Dec 9, 2018

Уже лучше, но все еще проблемы со шрифтами. У тебя сейчас с засечками, а на главной Яндекса - без засечек. И размеры шрифта почти везде не те. Можно ведь подглядеть на главной, какие они :) И взять оттуда логотип. А стрелку давай всё-таки чистой версткой, а не картинкой.

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