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

chore(Link): change text decoration on hover in 22 theme #3290

Merged
merged 2 commits into from
Oct 16, 2023

Conversation

HelenaIsh
Copy link
Contributor

@HelenaIsh HelenaIsh commented Oct 11, 2023

Проблема

В 22 теме подчеркивание в кнопках-ссылках и ссылках реализовано через border-bottom, а не text-decoratoion. Но иногда пользователям необходимо реализовать подчеркивание через text-decoratoion для однородности с другими элементами. Пример. Они отключают подчеркивание через border-bottom и ожидают, что и в ссылке и в кнопке-ссылке заработает подчеркивание через text-decoratoion. Проблема в том, что в кнопке оно не работает, а в ссылке работает.

Решение

Считаю что при отключении подчеркивания через border-bottom, text-decoratoion не должен автоматически включаться. Это касается и кнопки-ссылки и ссылки

Установила linkHoverTextDecoration = 'none' в 22х темах.

Объяснить пользователям, что это правильное поведение и в случае необходимости им самим нужно переопределять эту переменную.

Возможно завести проп, который переключает стили подчеркивания между border-bottom и text-decoratoion

Либо избавиться от border-bottom и использовать text-underline-offset - заведена задача IF-1490

Ссылки

fix IF-1472

Чек-лист перед запросом ревью

  1. Добавлены тесты на все изменения
    ⬜ unit-тесты для логики
    ⬜ скриншоты для верстки и кросс-браузерности
    ✅ нерелевантно

  2. Добавлена (обновлена) документация
    ⬜ styleguidist для пропов и примеров использования компонентов
    ⬜ jsdoc для утилит и хелперов
    ⬜ комментарии для неочевидных мест в коде
    ⬜ прочие инструкции (README.md, contributing.md и др.)
    ✅ нерелевантно

  3. Изменения корректно типизированы
    ✅ без использования any (см. PR 2856)
    ⬜ нерелевантно

  4. Прочее
    ✅ все тесты и линтеры на CI проходят
    ✅ в коде нет лишних изменений
    ✅ заголовок PR кратко и доступно отражает суть изменений (он попадет в changelog)

@HelenaIsh HelenaIsh marked this pull request as ready for review October 12, 2023 10:29
@HelenaIsh HelenaIsh requested a review from zhzz October 13, 2023 12:01
@HelenaIsh HelenaIsh changed the title chore(Button): change text decoration on hover in 22 theme chore(Link): change text decoration on hover in 22 theme Oct 16, 2023
@zhzz zhzz merged commit 1beb6da into master Oct 16, 2023
6 checks passed
@zhzz zhzz deleted the link-text-decoration-22 branch October 16, 2023 11:55
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.

2 participants