feat(tabs): add pre collapsed tab ids prop #413
Conversation
mrAnomalyy
requested review from
reme3d2y,
v-gevak,
EGNKupava and
SiebenSieben
as code owners
🦋 Changeset detectedLatest commit: 4ca7b9f The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Коллеги, доброе время суток. Хотелось бы узнать Ваше мнение об этих изменениях и понять, есть ли шансы смержить это в мастер? |
Pull Request Test Coverage Report for Build 3799223797
💛 - Coveralls |
|
Собрана новая демка. |
|
К задумке есть вопрос: если я скрываю первые табы в "ещё", то при выборе скрытого таба выбранный таб отображается первым в линии, что выглядит довольно странно и противоречит обычной логике скрытия табов в ещё. Обычно выбранный скрытй таб идет последним в линии. Перед доработкой нужно обсудить с дизайнерами, которые заказали правку. Мое мнение — скрывабельные табы должны идти после видимых. Чтобы механика была идентичной. Можно отправить дизайнера ко мне если что)
|
|
@Oladii у нас на проекте, к сожалению, нет дизайнеров. Идея была предложена командой фронтов (в т.ч. мной). Замечание хорошее, сегодня доделаю :) |
|
Собрана новая демка. |
|
Добавил дополнительную сортировку заголовков табов по наличию в массиве collapsedTabsIds. @Oladii |
|
Собрана новая демка. |
|
Собрана новая демка. |
| const elements = Array.from(container.querySelectorAll(selectors)) as HTMLElement[]; | ||
| const containerWidth = | ||
| (inlineSize || container.clientWidth) - (moreElementRect?.width || 0) * 1.5; // при рассчётах, даём кнопке "Ещё" чуть больше места, чтобы точно влезла |
There was a problem hiding this comment.
Изменил код здесь, посмотрите. Убрал одно магическое число и добавил другое - но с пояснением :))
|
Собрана новая демка. |
Опишите проблему
Наша команда хочет давать пользователям возможность выбора вкладок к отображению, в то время как остальные вкладки будут спрятаны под меню "Ещё".
Шаги для воспроизведения
Ожидаемое поведение
Выбранные вкладки заранее, вне зависимости ширины контейнера, будут спрятаны под меню "Ещё"
Чек лист
Тестовый стенд
Десктоп (если данных нет оставте блок пустым):
Дополнительная информация
В ПРе есть только одно изменение, которое влияет на поведение компонента в обычном сценарии использования, без нового пропа - добавлен учёт кнопки "Ещё" при расчёте ширины контейнера и определения списка вкладок, которые должны быть спрятаны в список "Ещё". В коде:
packages/tabs/src/hooks/use-collapsible-elements.ts:29