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

feat(hooks):add block body scroll hook #3405

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

veselov-as
Copy link

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

Добавлен хук useBodyScrollBlock - для отключения скрола на body, при открытии модальных окон

При открытии компонентов Modal или Sidebar, не блокируется скрол на body

Из-за этого, когда в Modal (Sidebar) компоненте, много контента и появляется внутренний скрол, можно проскролить контент не там где ожидается

Демка -
https://codesandbox.io/p/sandbox/relaxed-blackburn-mzs7sy?file=%2Fsrc%2FModal.tsx%3A11%2C37

Чек-лист

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

Опционально

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

};

export const useBlockBodyScroll = () => {
useLayoutEffect(() => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Привет, поигрался с примером, давай добавим в useBlockBodyScroll флаг вкл/выкл. Просто не всегда нужно заблокировать скрол у боди при монтировании компонента, иногда это нужно сделать по флагу.

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