Убрать `!important` из btn-transparent

В базовом коде Discourse используется !important. Уберите это, чтобы я мог переопределить стили без необходимости использовать !important у себя. Кнопки должны иметь состояния фокуса и наведения для доступности.

Привет! Эти прозрачные кнопки остаются прозрачными, так как это их ожидаемое поведение. Можешь указать, где именно ты пытаешься изменить эту функциональность? Кнопка остаётся прозрачной при наведении курсора и у тебя сейчас?

Да, они принуждают к прозрачности даже при наведении и в активном состоянии. Нам не нужны прозрачные кнопки; мы хотим, чтобы кнопки имели состояния наведения/фокуса и активного состояния.

На скриншоте показано расположение кнопки, но это верно для всех кнопок:

Я вернул кольцо фокуса, но ожидаю, что также будут изменяться цвет фона и цвет иконки, чтобы пользователь мог определить, когда курсор находится над кнопкой.

Цвет иконки действительно меняется при наведении, но фон не изменится, так как это прозрачная кнопка.

Я подключу к обсуждению дополнительных участников. Возможно, нам потребуется внести изменения.

Мы хотим, чтобы фон и цвет менялись, поэтому использование important ограничивает нас. Я также стараюсь никогда не использовать important в коде, если это абсолютно необходимо, поэтому предпочёл бы не бороться с переопределением базового кода.

1 лайк

Похоже, что удаление !important здесь допустимо. Я открыл запрос на слияние (pull request) по этой теме: DEV: remove !important from btn-transparent background by awesomerobot · Pull Request #29809 · discourse/discourse · GitHub

5 лайков