Skip to content

Fix stuck while loop where the checked element never gets updated #1140

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

Merged
merged 2 commits into from
Jul 2, 2025

Conversation

madsrasmussen
Copy link
Contributor

The TipTap Table Popover no longer opens after changes to the popover container. It gets stuck in the while loop because the checked element has static positioning. The element in the loop is never updated, so it keeps checking the same element repeatedly.

@madsrasmussen madsrasmussen changed the title Fix stuck while loop where the checked element never got updated Fix stuck while loop where the checked element never gets updated Jul 2, 2025
Copy link

github-actions bot commented Jul 2, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-beach-055ecb503-1140.westeurope.azurestaticapps.net

Copy link

sonarqubecloud bot commented Jul 2, 2025

@nielslyngsoe nielslyngsoe marked this pull request as ready for review July 2, 2025 18:17
Copy link

github-actions bot commented Jul 2, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-beach-055ecb503-1140.westeurope.azurestaticapps.net

@nielslyngsoe nielslyngsoe enabled auto-merge (squash) July 2, 2025 18:19
@nielslyngsoe nielslyngsoe disabled auto-merge July 2, 2025 18:19
@nielslyngsoe nielslyngsoe merged commit 3ae601b into main Jul 2, 2025
11 of 12 checks passed
@nielslyngsoe nielslyngsoe deleted the v1/bugfix/popover-container-stuck-in-while-loop branch July 2, 2025 18:45
@iOvergaard iOvergaard added the bug Something isn't working label Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants