-
Notifications
You must be signed in to change notification settings - Fork 691
Обновляет демки в backdrop
#5900
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
base: main
Are you sure you want to change the base?
Conversation
|
С адаптивом и переосмыслением согласна! А вот кнопка во всплывающем окне должна быть белая :) |
Покрасил в белый 💪 |
Inventoris
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Привет-привет)
Смотри, у нас были синие кнопки (основной цвет раздела CSS), белые всплывающие плашки. Всё как по стайлгайду (почти). Давай останемся тут в рамках стилей демок в Доке и сделаем вообще идеально:
- Обе демки оформим с синими кнопками, потому что розовый это дополнительный, когда синий уже есть и его нельзя использовать.
- Плашки сделаем белыми, вот пример. С учетом комментария Светы =)
Co-authored-by: Alexey Nikitchenko <[email protected]>
|
@Inventoris, заглянул ещё раз в стайлгайд:
*наверное, именно по этой причине Света сказала, что кнопка должна быть белой... ну вот, сам себя и убедил 😅 @skorobaeus, а вот про вытянутость диалоговых окон: мне кажется, что они выглядят как-то несуразно:
Опять таки для маленьких элементов хорошо выглядит, а для больших — уже не очень. Что думаешь? |
cfa9d8c to
9cc82d8
Compare
|
@skorobaeus, а стилей для белых кнопок нет в стайлгайде 🥹 Для фокуса покрасил бордер в основной цвет раздела: А для наведения кнопку сделал темной, а бордер с текстом белыми: Что думаешь? |
Превью контента из 9cc82d8 опубликовано. |
Ну по первому пункту могу сказать, что если эмодзи не подходит к кнопке, то лучше всё таки менять эмодзи, чем отклоняться от стайлгайда по цветам кнопок) А по второму тут лучше Света подскажет. Мне кажется стоило бы действительно обновить стайлгайд, чтобы не оставалось сомнений. Но пока лучше делать, как в других доках, а там крашено в белый) Если правила нет, или оно непонятное, лучше делать, как уже где-то сделано. |
Ну, не соглашусь 🙅♀️ Это скорее повод обсудить как правильно 😅 |
Они не кажутся мне несуразными, это вкусовщина. Но ты абсолютно прав насчёт того, что когда я писала стайлгайд, я попросту не учла модальные окна 😅 И вот эти белые контрастные edgy всплывашки хорошо работают только пока маленькие. По большому счёту нет никакой причины стилизовать полноценные модалки иначе, чем аналогичные блоки / формы в "основном теле" демки. Нужно тогда сформулировать как мы их будем стилизовать by default и зафиксировать это в дизайн-системе. У тебя уже есть какое-то соображение на этот счёт? |


Описание
https://content-5900.dev.doka.guide/css/backdrop/
Немного добавил адаптива, приглушил цвета и переосмыслил вторую демку: хотел показать пример на поповере
╰(*°▽°*)╯
Первая демка
Вторая демка