ayo/lion
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: resolve registrationComplete via microtask to be sync

Browse source
This commit is contained in:
Thomas Allmer 2020-07-28 13:58:13 +02:00
parent 4715b6b37d
commit 6b8daef509
2 changed files with 2 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
packages/form-core/src/choice-group/ChoiceGroupMixin.js
View file

@ -94,10 +94,7 @@ export const ChoiceGroupMixin = dedupeMixin(
connectedCallback() {
super.connectedCallback();
this.__registrationCompleteTimer = setTimeout(() => {
this.__resolveRegistrationComplete();
});
Promise.resolve().then(() => this.__resolveRegistrationComplete());
this.registrationComplete.then(() => {
this.__isInitialModelValue = false;
@ -110,7 +107,6 @@ export const ChoiceGroupMixin = dedupeMixin(
super.disconnectedCallback();
}
clearTimeout(this.__registrationCompleteTimer);
if (this.registrationComplete.done === false) {
this.__rejectRegistrationComplete();
}

6
packages/form-core/src/form-group/FormGroupMixin.js
View file

@ -154,10 +154,7 @@ export const FormGroupMixin = dedupeMixin(
connectedCallback() {
super.connectedCallback();
this.setAttribute('role', 'group');
this.__registrationCompleteTimer = setTimeout(() => {
this.__resolveRegistrationComplete();
});
Promise.resolve().then(() => this.__resolveRegistrationComplete());
this.registrationComplete.then(() => {
this.__isInitialModelValue = false;
@ -175,7 +172,6 @@ export const FormGroupMixin = dedupeMixin(
document.removeEventListener('click', this._checkForOutsideClick);
this.__hasActiveOutsideClickHandling = false;
}
clearTimeout(this.__registrationCompleteTimer);
if (this.registrationComplete.done === false) {
this.__rejectRegistrationComplete();
}