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

[intensive][HTML2] Правки и предложения по макету Кэт энерджи #1122

Open
dasha326 opened this issue Aug 16, 2023 · 3 comments

Comments

@dasha326
Copy link

Ссылка

https://www.figma.com/file/GLlcDqnmpzr82PtaPLHcIZ/HTML-2-%2F-%D0%9A%D1%8D%D1%82-%D1%8D%D0%BD%D0%B5%D1%80%D0%B4%D0%B6%D0%B8-(29)?type=design&node-id=94552-3944&mode=design&t=JYVm96BsbLyn9np7-0

Проблема или вопрос

Здравствуйте, это снова я с предложениями и правками.

  1. Высота кнопок:
    У формы - 52px
    На главной странице - 54px
    В каталоге - 40px
    Межстрочный интервал и шрифты тоже разнятся в некоторых местах. У кнопки формы вообще не написан межстрочный интервал.
    Былобы неплохо сделать кнопки одинаковыми. Или два варианта большая и маленькая.
  2. Везде разный line-height, у некоторого текста (Например: на первом экране у текста или у всех заголовков) даже встречается 1 к 1. Было бы лучше сделать какой-то стандратный line-height. Например заголовокам 1.2, а тексту 1.3 или 1.5. Чтобы не задавать каждому тексту во всем пороекте отдельный line-height.
  3. Меня немного смутили названия, воможно у студентов также. Где-то pressed, где-то down. А на деле это фокус или эктив состояние?
    image
  4. Заголовки у карточки товара. Почему бы не сделать грамовку всегда с новой строки? Потому что на мобильной версии в таком случае при переполнении, будет узкий высокий заголовок, ибо его нужно будет ограничивать по ширине.
    image
  5. У некоторых карточек товаров не хватает рубля в цене
  6. Добавить палитру цветов в styleguid (было бы неплохо во все макеты такое сделать), чтобы люди сразу видели все цвета которые есть в макете. А также все размеры и межстрочный интервал заголовков от h1 до h6.

Информация об устройстве

  • Девайс:
  • ОС:
  • Браузер:
  • Версия:

Дополнительная информация

No response

Возможное решение или предложение по улучшению

No response

@nikolai-shabalin
Copy link
Contributor

Привет, ещё раз спасибо.

  1. Будет два вариант в каталоге и на остальных страницах
  2. Обдумаем как улучшить
  3. Переделаем. Руссифицируем
  4. Не очень хороший вариант, так как может быть одно слово на второй строке от названия товара, а на третьей уже граммовка. Пока оставляем как есть
  5. Кажется починили
  6. Хорошая идея. Пока положим в бэклог, так как хочется сделать систему, а не просто взять цвета макета и сделать их переменными

@dasha326
Copy link
Author

dasha326 commented Aug 18, 2023

4.Не очень хороший вариант, так как может быть одно слово на второй строке от названия товара, а на третьей уже граммовка. Пока оставляем как есть
Но возможно тогда лучше растянуть заголовок от края до края, а не ограничивать его совсем на мобильных устройствах. И еще добавить какое-нибудь название подлиннее, потому что в реальной жизни одинаковые по длине заголовки редко встречаются.

@nikolai-shabalin
Copy link
Contributor

4.Не очень хороший вариант, так как может быть одно слово на второй строке от названия товара, а на третьей уже граммовка. Пока оставляем как есть Но возможно тогда лучше растянуть заголовок от края до края, а не ограничивать его совсем на мобильных устройствах. И еще добавить какое-нибудь название подлиннее, потому что в реальной жизни одинаковые по длине заголовки редко встречаются.

Хороший совет. Так и сделаем

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

No branches or pull requests

2 participants