fix(Button): add color to border when use=text
#3216
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Проблема
Отсутствует цвет обводки в состоянии
use=text
. Из-за чего в состоянияхhover
иactive
кнопка кажется меньше остальных.Решение
btnTextActiveBg
, но это ломающее изменение)buttonActiveMixin
как в других стилях кнопки, но вместо цвета обводки передаю цвет фонаВ других стилях кнопки есть отельные переменные для состояния
active
:btn...ActiveBg
btn...ActiveShadow
btn...ActiveBorderColor
btn...ActiveBorderTopColor
У стиля text есть только
btnTextActiveBg
. Остальные решил на добавлять, т.к. думаю концептуально у этого стиля фон всегда должен быть одного цвета.Ссылки
IF-1347
Чек-лист перед запросом ревью
Добавлены тесты на все изменения
⬜ unit-тесты для логики
✅ скриншоты для верстки и кросс-браузерности
⬜ нерелевантно
Добавлена (обновлена) документация
⬜ styleguidist для пропов и примеров использования компонентов
⬜ jsdoc для утилит и хелперов
⬜ комментарии для неочевидных мест в коде
⬜ прочие инструкции (
README.md
,contributing.md
и др.)✅ нерелевантно
Изменения корректно типизированы
⬜ без использования
any
(см. PR2856
)✅ нерелевантно
Прочее
✅ все тесты и линтеры на CI проходят
✅ в коде нет лишних изменений
✅ заголовок PR кратко и доступно отражает суть изменений (он попадет в changelog)