Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: useBodyScrollLock type error #1462

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

hrubysi
Copy link
Contributor

@hrubysi hrubysi commented Nov 27, 2024

Fixes type error of useBodyScrollLock introduced in #1428.

TypeError: Argument 1 ('element') to Window.getComputedStyle must be an instance of Element.

Apparently, not every target of touchmove event has to be Element. Also good example of why type casting should be avoided.

@hrubysi hrubysi force-pushed the fix/body-scroll-lock-type-error branch from 363d2d2 to ad7bd92 Compare November 28, 2024 09:12
Copy link
Member

@zernonia zernonia left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @hrubysi !

@zernonia zernonia merged commit 9c9732b into unovue:main Dec 2, 2024
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants