VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard

[Feature][FocusTrap]: Экспортировать FocusTrap

Open eugpoloz opened this issue 2 years ago • 4 comments

Сейчас FocusTrap — внутренний компонент библиотеки, наружу мы его не отдаем. Возможно, есть смысл это изменить.

### Что надо сделать
- [ ] Экспортировать `FocusTrap`
- [ ] Написать для него документацию

UPD

Прежде чем экспортировать нужно исследовать насколько наш FocusTrap правилен с точки зрения реализации и API.

Как референс можно сравнить с https://floating-ui.com/docs/FloatingFocusManager

eugpoloz avatar May 18 '23 12:05 eugpoloz

А были ли запросы на экспорт?

PS: подумывал заюзать решение из либы https://floating-ui.com/docs/FloatingFocusManager

inomdzhon avatar May 18 '23 12:05 inomdzhon

@inomdzhon

А были ли запросы на экспорт?

После BOF-сессии на HolyJS были. 😉

PS: подумывал заюзать решение из либы https://floating-ui.com/docs/FloatingFocusManager

Пролистала быстро, не вижу там настройки для возврата фокуса назад после закрытия floating-элемента. Или это происходит автоматически и настраивать не надо?

А чем наш FocusTrap не устраивает, чего не хватает? Или с учетом Floating UI проще их же решение затащить.

eugpoloz avatar May 18 '23 20:05 eugpoloz

Пролистала быстро, не вижу там настройки для возврата фокуса назад после закрытия floating-элемента. Или это происходит автоматически и настраивать не надо?

Вот этот параметр https://floating-ui.com/docs/floatingfocusmanager#returnfocus

А чем наш FocusTrap не устраивает, чего не хватает? Или с учетом Floating UI проще их же решение затащить.

Второе) Использовать уже отработанное решение может быть эффективней

С другой стороны, FocusTrap это утилитарный компонент, не уверен, что правильно его отдавать прямо через пакет @vkontakte/vkui, возможно это должен быть иной самодостаточный пакет с утилитами

inomdzhon avatar May 19 '23 08:05 inomdzhon

Есть запрос от нас еще

egorprnn avatar Feb 09 '24 15:02 egorprnn