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]: DOM changes will freeze page #461

Closed
chasegiunta opened this issue Mar 24, 2023 · 1 comment
Closed

[Bug]: DOM changes will freeze page #461

chasegiunta opened this issue Mar 24, 2023 · 1 comment
Labels
🛑 Abandoned The issue or Pull Request will not be worked on 🐞 Bug Bug or Bug fixes

Comments

@chasegiunta
Copy link

Reproduction

Not really a reproduction, but pointer is showing (

for (const { type, target } of records) {
) where javascript profiler (this is minified code in production, apologies) is indicating the performance issue is.

image

Describe the bug

It's difficult for me to accurately describe which elements, or how many, are appearing on screen, but because we have many DOM elements that are lazily loaded based on scroll usage, the mutation observer callback is firing off like crazy. I'm wondering if debouncing or throttling the function can prevent this.

Severity

blocking all usage of Twind

System Info

## Binaries:
 - Node: 14.18.2 - ~/.nvm/versions/node/v14.18.2/bin/node
 - Yarn: 3.4.1 - ~/.nvm/versions/node/v14.18.2/bin/yarn
 - npm: 6.14.15 - ~/.nvm/versions/node/v14.18.2/bin/npm
 - Watchman: 2022.11.14.00 - /opt/homebrew/bin/watchman
## Browsers:
 - Chrome: 111.0.5563.110
 - Firefox: 108.0.2
 - Safari: 16.3
## npmPackages:
 - @twind/core: ^1.1.3 => 1.1.3 
 - @twind/preset-tailwind: ^1.1.4 => 1.1.4

Additional Information

No response

@chasegiunta chasegiunta added the 🐞 Bug Bug or Bug fixes label Mar 24, 2023
@stale stale bot added the 🛑 Abandoned The issue or Pull Request will not be worked on label Dec 20, 2023
Copy link

stale bot commented Jan 19, 2024

Hey folks. This issue hasn't received any traction for 90 days, so we're going to close this for housekeeping. If this is still an ongoing issue, please do consider contributing a Pull Request to resolve it. Further discussion is always welcome even with the issue closed. If anything actionable is posted in the comments, we'll consider reopening it.

@stale stale bot closed this as completed Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛑 Abandoned The issue or Pull Request will not be worked on 🐞 Bug Bug or Bug fixes
Projects
None yet
Development

No branches or pull requests

1 participant