-
Notifications
You must be signed in to change notification settings - Fork 75
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
fix(input, input-number, input-text): Fix infinite loop crashing browser. #5882 #5961
Conversation
@driskull these changes look good, about the same as what I have (so far) come up with for resolving the infinite loop bug, but I still don't know how to get the |
Yeah, I think something with focus-trap isn't allowing this. Maybe we can do this via a follow-up. I will add a way to disable the focus trap for Modal. |
@eriklharper can we get this one installed today? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eriklharper can we get this one installed today?
Yes, let's do it. Approved :)
@driskull Do you have an open PR for the |
Yeah here's the PR: #5965 |
* master: (55 commits) build: update browserslist db (#5986) docs: update component READMEs (#5980) 1.0.0-next.670 refactor(input-date-picker,date-picker)!: Removing deprecated locale properties (#5977) 1.0.0-next.669 refactor(input-date-picker)!: Remove deprecated active property (#5974) 1.0.0-next.668 fix(modal, popover): Add `disableFocusTrap` property to toggle focus trapping. (#5965) 1.0.0-next.667 refactor(input-time-picker)!: Remove deprecated active property (#5970) docs(changelog): fix breaking change indent levels (#5973) 1.0.0-next.666 refactor(time-picker)!: Remove deprecated locale property (#5962) 1.0.0-next.665 fix(input, input-number, input-text): Fix infinite loop crashing browser. #5882 (#5961) test(floating-ui): fix type errors (#5966) docs: update component READMEs (#5964) 1.0.0-next.664 fix(alert): auto-dismissible retains close button and dismisses timer while a user is hovering over (#5872) chore(color-picker): add opacity string (#5959) ...
Related Issue: #5882
Summary