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

[Bug]: На стрелке в HorizontalScroll не работает прокрутка колесом #7774

Closed
SevereCloud opened this issue Oct 16, 2024 · 4 comments · Fixed by #7882

Comments

@SevereCloud
Copy link
Contributor

SevereCloud commented Oct 16, 2024

Описание

При наведении на стрелку в HorizontalScroll не работает прокрутка через колесо мыши

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

Шаги воспроизведения

  1. Навести мышку на стрелку в скроле
  2. Попытаться проскролить

Ожидаемое поведение

Прокрутка скрола

Скриншоты

2024-10-16.16.06.42.mov

Пример с воспроизведением

HorizontalScroll

@vkcom-publisher
Copy link
Contributor

v6.7.4 🎉

@vkcom-publisher
Copy link
Contributor

v7.0.0-beta.2 🎉

@inomdzhon inomdzhon modified the milestones: v7.0.0-beta.2, v7.0.0 Dec 2, 2024
@vkcom-publisher
Copy link
Contributor

v7.0.0 🎉

inomdzhon added a commit that referenced this issue Feb 19, 2025
В **Firefox** событие `onwheel` на контейнере и одновременная прокрутка скроллируемого элемента (`scrollBy` или `scrollLeft`) **тач-падом** негативно влияет на перерисовку контента во время скролла.

Чтобы решить проблему #7774, можно навесить `onwheel` на каждую из стрелок навигации, но у этого решения есть свой баг: если начать крутить колесо с сначала или конца, то появление стрелки навигации заблоркирует прокрутку, то тех пор пока не двинешь мышкой над ней, чтобы началась обработка события `onwheel` на стрелке.

Из-за этих проблем оптимальным будет откатить фикс #7774 и пометить его как **Won't Fix**. Это будет являться ограничением компонента.

Тестировалось на версиях **Firefox** `134` и `135`.

Наблюдается с `v6.7.4`.

- reverts #7882, #8177
inomdzhon added a commit that referenced this issue Feb 19, 2025
- reverts #7882, #8177

h2. Описание

В **Firefox** событие `onwheel` на контейнере и одновременная прокрутка скроллируемого элемента (`scrollBy` или `scrollLeft`) **тач-падом** негативно влияет на перерисовку контента во время скролла.

Чтобы решить проблему #7774, можно навесить `onwheel` на каждую из стрелок навигации, но у этого решения есть свой баг: если начать крутить колесо с сначала или конца, то появление стрелки навигации заблоркирует прокрутку, то тех пор пока не двинешь мышкой над ней, чтобы началась обработка события `onwheel` на стрелке.

Из-за этих проблем оптимальным будет откатить фикс #7774 и пометить его как **Won't Fix**. Это будет являться ограничением компонента.

Тестировалось на версиях **Firefox** `134` и `135`.

Наблюдается с `v6.7.4` до `v7.1.1` при передаче `scrollOnAnyWheel`, а с `v7.1.2` всегда (связано с #8177).

/~https://github.com/user-attachments/assets/008907de-e6bd-4807-bbea-b0e7f69a8605

h2. Release notes
h2. Исправления
- HorizontalScroll: отменён фикс #7774 из-за проблем перерисовок в **Firefox** при прокрутке через тач-пад, теперь невозможность прокрутить колесом мыши над стрелкой навигации будет считаться ограничением компонента
inomdzhon added a commit that referenced this issue Feb 19, 2025
- reverts #7882, #8177

h2. Описание

В **Firefox** событие `onwheel` на контейнере и одновременная прокрутка скроллируемого элемента (`scrollBy` или `scrollLeft`) **тач-падом** негативно влияет на перерисовку контента во время скролла.

Чтобы решить проблему #7774, можно навесить `onwheel` на каждую из стрелок навигации, но у этого решения есть свой баг: если начать крутить колесо с сначала или конца, то появление стрелки навигации заблоркирует прокрутку, то тех пор пока не двинешь мышкой над ней, чтобы началась обработка события `onwheel` на стрелке.

Из-за этих проблем оптимальным будет откатить фикс #7774 и пометить его как **Won't Fix**. Это будет являться ограничением компонента.

Тестировалось на версиях **Firefox** `134` и `135`.

Наблюдается с `v6.7.4` до `v7.1.1` при передаче `scrollOnAnyWheel`, а с `v7.1.2` всегда (связано с #8177).

/~https://github.com/user-attachments/assets/008907de-e6bd-4807-bbea-b0e7f69a8605

h2. Release notes
h2. Исправления
- HorizontalScroll: отменён фикс #7774 из-за проблем перерисовок в **Firefox** при прокрутке через тач-пад, теперь невозможность прокрутить колесом мыши над стрелкой навигации будет считаться ограничением компонента
inomdzhon added a commit that referenced this issue Feb 19, 2025
)

- reverts #7882, #8177

h2. Описание

В **Firefox** событие `onwheel` на контейнере и одновременная прокрутка скроллируемого элемента (`scrollBy` или `scrollLeft`) **тач-падом** негативно влияет на перерисовку контента во время скролла.

Чтобы решить проблему #7774, можно навесить `onwheel` на каждую из стрелок навигации, но у этого решения есть свой баг: если начать крутить колесо с сначала или конца, то появление стрелки навигации заблоркирует прокрутку, то тех пор пока не двинешь мышкой над ней, чтобы началась обработка события `onwheel` на стрелке.

Из-за этих проблем оптимальным будет откатить фикс #7774 и пометить его как **Won't Fix**. Это будет являться ограничением компонента.

Тестировалось на версиях **Firefox** `134` и `135`.

Наблюдается с `v6.7.4` до `v7.1.1` при передаче `scrollOnAnyWheel`, а с `v7.1.2` всегда (связано с #8177).

/~https://github.com/user-attachments/assets/008907de-e6bd-4807-bbea-b0e7f69a8605

h2. Release notes
h2. Исправления
- HorizontalScroll: отменён фикс #7774 из-за проблем перерисовок в **Firefox** при прокрутке через тач-пад, теперь невозможность прокрутить колесом мыши над стрелкой навигации будет считаться ограничением компонента
@inomdzhon
Copy link
Contributor

inomdzhon commented Feb 19, 2025

Фикс отменём с v7.1.3.

см. описание #8294

@inomdzhon inomdzhon closed this as not planned Won't fix, can't repro, duplicate, stale Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants