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

refactor(Tabs): used ResizeObserver to detect tab width changes #1366

Merged
merged 2 commits into from
Jul 23, 2021

Conversation

nekitk
Copy link
Collaborator

@nekitk nekitk commented Jul 22, 2021

Описание изменений

  • Переписал механизм расчёта положения подчёркивания в табах с использованием ResizeObserver
  • Упростил вёрстку подчёркивания
  • Добавил хук useResizeObserved, который позволяет отслеживать изменение размеров сразу группы элементов

Чек-лист

  • PR: направлен в правильную ветку
  • PR: назначен исполнитель PR и указаны нужные лейблы
  • PR: проверен diff, ничего лишнего в PR не попало
  • PR: прилинкованы затронутые issue и связанные PR
  • PR: есть описание изменений
  • JS: нет варнингов и ошибок в консоли браузера
  • Тесты: новый функционал и исправленные баги покрыты тестами
  • Документация: отражены все изменения в API компонентов и описаны важные особенности реализации или использования
  • Сторибук: для компонентов написаны или обновлены stories
  • Верстка: используются переменные
  • Верстка: проверена с разным количеством контента

Опционально

  • Доработки: заведены задачи для дальнейшей работы, если что-то решено не править в текущем пулл-реквесте
  • Коммиты: проименованы в соответствии с правилами

@nekitk nekitk requested review from gizeasy and al-fyodorov July 22, 2021 16:43
@nekitk nekitk self-assigned this Jul 22, 2021
@vercel
Copy link

vercel bot commented Jul 22, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/consta/consta-uikit/AjaCqRUQaHbxDhvHWUyzqXo14145
✅ Preview: https://consta-uikit-git-refactor-tabs-consta.vercel.app

) {
if (lineRef.current) {
const lineStyle = lineRef.current.style;
lineStyle.setProperty('--tabsWidth', `${tabsWidth}px`);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦

@gizeasy gizeasy merged commit edbb728 into dev Jul 23, 2021
@gizeasy gizeasy deleted the refactor/tabs branch July 23, 2021 08:14
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 this pull request may close these issues.

3 participants