From 0384ae79f8895fa2fc61ef6e877d799af84ad55b Mon Sep 17 00:00:00 2001 From: nd0ut Date: Mon, 19 Feb 2024 18:22:22 +0300 Subject: [PATCH] fix(shadow-wrapper): wait for `shadowStyles` load before calling `shadowReadyCallback` --- blocks/ShadowWrapper/ShadowWrapper.js | 30 ++++++++++++++++++++++++++- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/blocks/ShadowWrapper/ShadowWrapper.js b/blocks/ShadowWrapper/ShadowWrapper.js index a43c1d4af..2127edf5e 100644 --- a/blocks/ShadowWrapper/ShadowWrapper.js +++ b/blocks/ShadowWrapper/ShadowWrapper.js @@ -23,6 +23,33 @@ export function shadowed(Base) { shadowReadyCallback() {} + /** + * @private + * + * This is super tricky workaround to wait for shadow styles to load. When `shadowStyles` is defined, symbiote will + * wait for it to load before rendering the component. So the `render` method becomes async. We need to call + * `shadowReadyCallback` right after the rendering is done. But we can't just call it after `render` because it's + * async. So we need to wait for the shadow styles to load and then call `shadowReadyCallback`. + */ + async _waitForShadowStylesLoad() { + if (!this.shadowRoot) { + return; + } + // @ts-expect-error + const shadowStylesUrl = this.constructor['__shadowStylesUrl']; + /** @type {HTMLLinkElement | null} */ + const link = this.shadowRoot.querySelector(`link[href="${shadowStylesUrl}"]`); + + if (!link) { + return; + } + + await new Promise((resolve, reject) => { + link.addEventListener('load', resolve); + link.addEventListener('error', reject); + }); + } + initCallback() { super.initCallback(); this.setAttribute('hidden', ''); @@ -41,8 +68,9 @@ export function shadowed(Base) { link.onload = () => { // CSS modules can be not loaded at this moment // TODO: investigate better solution - window.requestAnimationFrame(() => { + window.requestAnimationFrame(async () => { this.render(); + await this._waitForShadowStylesLoad(); window.setTimeout(() => { this.removeAttribute('hidden'); this.shadowReadyCallback();