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

Link: pseudo #148

Closed
MaksimAliakhnovich opened this issue Jul 13, 2022 · 0 comments · Fixed by #156
Closed

Link: pseudo #148

MaksimAliakhnovich opened this issue Jul 13, 2022 · 0 comments · Fixed by #156
Assignees

Comments

@MaksimAliakhnovich
Copy link

Описание

Предлагаю доработать пропс pseudo.
Как сейчас: красивая картинка, но без возможности выбора через таб + проблемы с линтом.
image

Поведение

  • По хорошему в режиме pseudo ссылка не может быть тегом a и должна быть button. Да, сейчас можно прокинуть Component='button', тогда у кнопки вылезают стили дефолтной кнопки
    image
    , которые нужно вручную убирать. Но это ладно ещё.
  • Но также срабатывает алерт от линта на правило anchor-is-valid, а это уже исправить можно только отключением правила в строке. Тут видимо нужно пофиксить типы.
    https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/1c06306ede29bab6b0cdc1915dd6dcd6e847356b/docs/rules/anchor-is-valid.md
  • и что хуже, и уже затрагивает пользователей, когда ссылка pseudo в ней обычно нет href, потому что он не надо, но тогда такой элемент не получает фокус, и его нельзя выбрать через таб. Что в принципе хотелось бы видеть для интерактивного элемента. Как и outline, который тут тоже не предусмотрен
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants