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

fix(side-panel): component ref scroll on mobile version #540

Merged
merged 1 commit into from
Feb 20, 2023

Conversation

reabiliti
Copy link
Contributor

Опишите проблему

В мобильной версии сайд панели не используется обертка в виде Drawer, где проставляется owerflow: auto; для BaseModal и получается что при попытке скролить через рефу componentRef ничего не происходит, потому-что скрол ставится на враппере.

@changeset-bot
Copy link

changeset-bot bot commented Feb 16, 2023

🦋 Changeset detected

Latest commit: 4d6e25c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 21 packages
Name Type
@alfalab/core-components-base-modal Patch
@alfalab/core-components-bottom-sheet Patch
@alfalab/core-components-modal Patch
@alfalab/core-components-navigation-bar Patch
@alfalab/core-components-side-panel Patch
@alfalab/core-components-drawer Patch
@alfalab/core-components-gallery Patch
@alfalab/core-components-select Patch
@alfalab/core-components-tooltip Patch
@alfalab/core-components-calendar Patch
@alfalab/core-components-input-autocomplete Patch
@alfalab/core-components-intl-phone-input Patch
@alfalab/core-components-picker-button Patch
@alfalab/core-components-select-with-tags Patch
@alfalab/core-components-table Patch
@alfalab/core-components-calendar-input Patch
@alfalab/core-components-calendar-range Patch
@alfalab/core-components-calendar-with-skeleton Patch
@alfalab/core-components-date-range-input Patch
@alfalab/core-components-date-time-input Patch
@alfalab/core-components-tabs Patch

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

@coveralls
Copy link

coveralls commented Feb 16, 2023

Pull Request Test Coverage Report for Build 4221169060

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 79.888%

Totals Coverage Status
Change from base Build 4202848833: 0.0%
Covered Lines: 7594
Relevant Lines: 8629

💛 - Coveralls

@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@v-gevak
Copy link
Contributor

v-gevak commented Feb 17, 2023

@v-gevak
Copy link
Contributor

v-gevak commented Feb 17, 2023

Не хочется менять контейнер с overflow, потому что я уже на него завязался в #494:) Если влить этот ПР, то в моем все сломается. Может в реф просто скролл функцию передавать через useImperativeHandle? contentRef.current.scrollTo(pos)

@reabiliti
Copy link
Contributor Author

Не хочется менять контейнер с overflow, потому что я уже на него завязался в #494:) Если влить этот ПР, то в моем все сломается. Может в реф просто скролл функцию передавать через useImperativeHandle? contentRef.current.scrollTo(pos)

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

@reabiliti
Copy link
Contributor Author

Screen.Recording.2023-02-17.at.10.14.50.AM.mov

вроде не нашел проблем с хедером, может что-то еще надо настроить?

@v-gevak
Copy link
Contributor

v-gevak commented Feb 17, 2023

вроде не нашел проблем с хедером, может что-то еще надо настроить?

Попробуй контента добавить
Подтяни свежий мастер плиз, ПР с модалками влили

@reabiliti
Copy link
Contributor Author

reabiliti commented Feb 17, 2023

вроде не нашел проблем с хедером, может что-то еще надо настроить?

Воспроизводится с десктопа. Подтяни свежий мастер плиз, ПР с модалками влили

Есть тогда возможность пофиксить, чтобы было одинаковое поведение как раньше для десктопа и мобилки? @SiebenSieben @Lacronts - я ухожу в отпуск и не смогу в ближ время заниматься задачами

Можете себе взять в беклог?

@v-gevak
Copy link
Contributor

v-gevak commented Feb 17, 2023

вроде не нашел проблем с хедером, может что-то еще надо настроить?

Воспроизводится с десктопа. Подтяни свежий мастер плиз, ПР с модалками влили

Есть тогда возможность пофиксить, чтобы было одинаковое поведение как раньше для десктопа и мобилки? @SiebenSieben @Lacronts - я ухожу в отпуск и не смогу в ближ время заниматься задачами

Можете себе взять в беклог?

Да, ок. Исправим

@v-gevak v-gevak force-pushed the fix/side-panel-component-ref-scroll-for-mobile branch from a47a441 to ef304d8 Compare February 20, 2023 06:53
@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@v-gevak v-gevak force-pushed the fix/side-panel-component-ref-scroll-for-mobile branch from ef304d8 to 4d6e25c Compare February 20, 2023 07:48
@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@v-gevak v-gevak merged commit 979da63 into master Feb 20, 2023
@v-gevak v-gevak deleted the fix/side-panel-component-ref-scroll-for-mobile branch February 20, 2023 09:17
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.

None yet

6 participants