Skip to content

Commit

Permalink
Applying changes from v6 to AdaptiveProvider (#6536) (#6585)
Browse files Browse the repository at this point in the history
Это черри пик merge коммита bf26ee0 из #6489 
Попытка перенести исправления в подсчёте `sizeY` у `AdoptivityProvider` из v6 в v5.

#6476) был именно для v5.

- в целом мы полностью переносим логику затрагивающую работу `AdoptivityProvider` без `bridge`.
В то же время логику c `bridge` мы не трогаем. В том числе оставляем *неявное* использование `_hasPointer` из `vkjs` и игнорирование `hasPointer` свойства `AdoptivityProvider`. Сделано специально, чтобы никак не затронуть пользователей bridge, ведь на них баг не распространяется.
https://github.com/VKCOM/VKUI/blob/f361335092b365360b3b8ee65c145fb6b621a414/packages/vkui/src/components/AdaptivityProvider/AdaptivityProvider.tsx#L105-L112
https://github.com/VKCOM/VKUI/blob/f361335092b365360b3b8ee65c145fb6b621a414/packages/vkui/src/components/AdaptivityProvider/AdaptivityProvider.tsx#L2

```js
export const hasMouse = /*#__PURE__*/ (() => detect.hasMouse)();
```
[hasMouse source](https://github.com/VKCOM/vkjs/blob/3ab9216c6c9f7f48998176e9403eaba7450b9a31/src/InputUtils.ts#L48)

Также я заметил, что есть специльный тест, котороый реагирует именно на `_hasPointer` проверку, так что оставил, в v6 мы всё равно это убираем.
https://github.com/VKCOM/VKUI/blob/f361335092b365360b3b8ee65c145fb6b621a414/packages/vkui/src/components/AdaptivityProvider/AdaptivityProvider.test.tsx#L300-L308
  • Loading branch information
mendrew authored Feb 16, 2024
1 parent 3c22e88 commit f662902
Show file tree
Hide file tree
Showing 3 changed files with 440 additions and 73 deletions.
Loading

0 comments on commit f662902

Please sign in to comment.