From 2abc7cd40f2ff4eb903878f2c6ed9fea30a45876 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Attila=20T=C3=B3th?= <attila.th@hotmail.com>
Date: Tue, 18 Jul 2023 10:21:33 +0200
Subject: [PATCH] fix: optimize css style with using adoptedStyleSheets

---
 src/option.js     |  65 +++++++------
 src/selectlist.js | 242 +++++++++++++++++++++-------------------------
 2 files changed, 146 insertions(+), 161 deletions(-)

diff --git a/src/option.js b/src/option.js
index 2e446b7..7fd6f68 100644
--- a/src/option.js
+++ b/src/option.js
@@ -1,42 +1,44 @@
 const template = document.createElement('template');
 
 template.innerHTML = /* html */`
-  <style>
-    :host {
-      display: block;
-      list-style: none;
-      line-height: revert;
-      white-space: nowrap;
-      white-space-collapse: collapse;
-      text-wrap: nowrap;
-      min-height: 1.2em;
-      padding: .25em;
-      font-size: .875em;
-    }
+  <slot></slot>
+`;
 
-    :host(:hover) {
-      background-color: selecteditem;
-      color: selecteditemtext;
-      cursor: default;
-      user-select: none;
-    }
+const optionStyles = new CSSStyleSheet();
+
+optionStyles.replaceSync(/* css */`
+:host {
+  display: block;
+  list-style: none;
+  line-height: revert;
+  white-space: nowrap;
+  white-space-collapse: collapse;
+  text-wrap: nowrap;
+  min-height: 1.2em;
+  padding: .25em;
+  font-size: .875em;
+}
 
-    :host([disabled]) {
-      pointer-events: none;
-      color: rgba(16, 16, 16, 0.3);
-    }
+:host(:hover) {
+  background-color: selecteditem;
+  color: selecteditemtext;
+  cursor: default;
+  user-select: none;
+}
 
-    :host(.\\:checked[disabled]) {
-      background-color: rgb(176, 176, 176);
-    }
+:host([disabled]) {
+  pointer-events: none;
+  color: rgba(16, 16, 16, 0.3);
+}
 
-    :host(:focus-visible) {
-      outline: -webkit-focus-ring-color auto 1px;
-    }
+:host(.\\:checked[disabled]) {
+  background-color: rgb(176, 176, 176);
+}
 
-  </style>
-  <slot></slot>
-`;
+:host(:focus-visible) {
+  outline: -webkit-focus-ring-color auto 1px;
+}
+`);
 
 class OptionElement extends globalThis.HTMLElement {
   static formAssociated = true;
@@ -53,6 +55,7 @@ class OptionElement extends globalThis.HTMLElement {
 
     this.attachShadow({ mode: 'open' });
     this.shadowRoot.append(template.content.cloneNode(true));
+    this.shadowRoot.adoptedStyleSheets = [optionStyles];
   }
 
   attributeChangedCallback(name, oldVal, newVal) {
diff --git a/src/selectlist.js b/src/selectlist.js
index 9a225fe..40ad11d 100644
--- a/src/selectlist.js
+++ b/src/selectlist.js
@@ -4,116 +4,110 @@ const popoverSupported = typeof HTMLElement !== 'undefined' &&
   'popover' in HTMLElement.prototype;
 
 const popoverStyles = popoverSupported ? '' : /* css */`
-  [popover] {
-    position: fixed;
-    z-index: 2147483647;
-    padding: .25em;
-    width: fit-content;
-    border: solid;
-    background: canvas;
-    color: canvastext;
-    overflow: auto;
-    margin: auto;
-  }
+[popover] {
+  position: fixed;
+  z-index: 2147483647;
+  padding: .25em;
+  width: fit-content;
+  border: solid;
+  background: canvas;
+  color: canvastext;
+  overflow: auto;
+  margin: auto;
+}
 
-  [popover]:not(.\\:popover-open) {
-    display: none;
-  }
-`;
+[popover]:not(.\\:popover-open) {
+  display: none;
+}
+`,
+  listboxStyles = /* css */`
+[behavior="listbox"] {
+  box-sizing: border-box;
+  margin: 0;
+  min-block-size: 1lh;
+  max-block-size: inherit;
+  min-inline-size: inherit;
+  inset: inherit;
+}
+`,
+  popoverStyleSheet = new CSSStyleSheet(),
+  listboxStyleSheet = new CSSStyleSheet(),
+  selectlistStyleSheet = new CSSStyleSheet(),
+  documentStyles = new CSSStyleSheet();
+
+popoverStyleSheet.replaceSync(popoverStyles);
+listboxStyleSheet.replaceSync(listboxStyles);
+
+selectlistStyleSheet.replaceSync(/* css */ `
+:host {
+  display: inline-block;
+  position: relative;
+  font-size: .875em;
+}
 
-const listboxStyles = /* css */`
-  [behavior="listbox"] {
-    box-sizing: border-box;
-    margin: 0;
-    min-block-size: 1lh;
-    max-block-size: inherit;
-    min-inline-size: inherit;
-    inset: inherit;
-  }
-`;
+[part="button"] {
+  display: inline-flex;
+  align-items: center;
+  background-color: field;
+  cursor: default;
+  appearance: none;
+  border-radius: .25em;
+  padding: .25em;
+  border-width: 1px;
+  border-style: solid;
+  border-color: rgb(118, 118, 118);
+  border-image: initial;
+  color: buttontext;
+  line-height: min(1.3em, 15px);
+}
+
+:host([disabled]) [part="button"] {
+  background-color: rgba(239, 239, 239, .3);
+  color: graytext;
+  opacity: .7;
+  border-color: rgba(118, 118, 118, .3);
+}
+
+[part="marker"] {
+  height: 1em;
+  margin-inline-start: 4px;
+  opacity: 1;
+  padding-bottom: 2px;
+  padding-inline-start: 3px;
+  padding-inline-end: 3px;
+  padding-top: 2px;
+  width: 1.2em;
+}
+
+[part="listbox"] {
+  box-sizing: border-box;
+  box-shadow: rgba(0, 0, 0, .13) 0px 12.8px 28.8px,
+    rgba(0, 0, 0, .11) 0px 0px 9.2px;
+  min-block-size: 1lh;
+  border-width: 1px;
+  border-style: solid;
+  border-color: rgba(0, 0, 0, .15);
+  border-image: initial;
+  border-radius: .25em;
+  padding: .25em 0;
+}
+`);
 
 /**
  * CSS @layer: any styles declared outside of a layer will override styles
  * declared in a layer, regardless of specificity.
  * https://developer.mozilla.org/en-US/docs/Web/CSS/@layer
  */
+documentStyles.replaceSync(/* css */ `
+@layer {
+  ${popoverStyles}
+  x-selectlist ${listboxStyles}
+`);
 
-const headTemplate = document.createElement('template');
-headTemplate.innerHTML = /* html */`
-<style>
-  @layer {
-    ${popoverStyles}
-    x-selectlist ${listboxStyles}
-  }
-</style>
-`;
-
-document.head.prepend(headTemplate.content.cloneNode(true));
+document.adoptedStyleSheets.push(documentStyles);
 
 const template = document.createElement('template');
-template.innerHTML = /* html */`
-  <style>
-    ${popoverStyles}
-    ${listboxStyles}
-
-    :host {
-      display: inline-block;
-      position: relative;
-      font-size: .875em;
-    }
-
-    [part="button"] {
-      display: inline-flex;
-      align-items: center;
-      background-color: field;
-      cursor: default;
-      appearance: none;
-      border-radius: .25em;
-      padding: .25em;
-      border-width: 1px;
-      border-style: solid;
-      border-color: rgb(118, 118, 118);
-      border-image: initial;
-      color: buttontext;
-      line-height: min(1.3em, 15px);
-    }
-
-    :host([disabled]) [part="button"] {
-      background-color: rgba(239, 239, 239, .3);
-      color: graytext;
-      opacity: .7;
-      border-color: rgba(118, 118, 118, .3);
-    }
-
-    [part="marker"] {
-      height: 1em;
-      margin-inline-start: 4px;
-      opacity: 1;
-      padding-bottom: 2px;
-      padding-inline-start: 3px;
-      padding-inline-end: 3px;
-      padding-top: 2px;
-      width: 1.2em;
-    }
-
-    slot[name="listbox"],
-    ::slotted([slot="listbox"]) {
-      ${/* min-inline-size overridden below by selectlist width */''}
-    }
-
-    [part="listbox"] {
-      box-sizing: border-box;
-      box-shadow: rgba(0, 0, 0, .13) 0px 12.8px 28.8px,
-        rgba(0, 0, 0, .11) 0px 0px 9.2px;
-      min-block-size: 1lh;
-      border-width: 1px;
-      border-style: solid;
-      border-color: rgba(0, 0, 0, .15);
-      border-image: initial;
-      border-radius: .25em;
-      padding: .25em 0;
-    }
-  </style>
+template.innerHTML = /* html */ `
   <slot name="button">
     <button part="button" behavior="button" aria-haspopup="listbox">
       <slot name="selected-value">
@@ -136,7 +130,8 @@ template.innerHTML = /* html */`
 class SelectListElement extends globalThis.HTMLElement {
   static formAssociated = true;
   static observedAttributes = ['disabled', 'required', 'multiple'];
-  #internals;
+  #internals;  
+  #inlineSizeStyle = new CSSStyleSheet();
 
   constructor() {
     super();
@@ -149,6 +144,18 @@ class SelectListElement extends globalThis.HTMLElement {
 
     this.attachShadow({ mode: 'open' });
     this.shadowRoot.append(template.content.cloneNode(true));
+    /* min-inline-size overridden below by selectlist width */
+    this.#inlineSizeStyle.insertRule(
+      'slot[name="listbox"], ::slotted([slot="listbox"]) {}',
+      0
+    );
+
+    this.shadowRoot.adoptedStyleSheets = [
+      popoverStyleSheet,
+      listboxStyleSheet,
+      selectlistStyleSheet,
+      this.#inlineSizeStyle,
+    ];
 
     this.addEventListener('click', this.#onClick, true);
     this.addEventListener('keydown', this.#onKeydown);
@@ -415,7 +422,7 @@ class SelectListElement extends globalThis.HTMLElement {
 
   #handleReposition = () => {
     if (this.#isOpen()) {
-      reposition(this, this.#listboxEl);
+      reposition(this, this.#listboxEl, this.#inlineSizeStyle);
     }
   }
 
@@ -436,7 +443,7 @@ class SelectListElement extends globalThis.HTMLElement {
       this.#listboxEl.classList.add(':popover-open');
     }
 
-    reposition(this, this.#listboxEl);
+    reposition(this, this.#listboxEl, this.#inlineSizeStyle);
 
     const activeOptions = this.options.filter(opt => !opt.disabled);
     const currentOption = this.selectedOption
@@ -470,10 +477,8 @@ class SelectListElement extends globalThis.HTMLElement {
   }
 }
 
-function reposition(reference, popover) {
-
-  let { style } = getCSSRule(reference.shadowRoot,
-    'slot[name="listbox"], ::slotted([slot="listbox"])');
+function reposition(reference, popover, inlineSizeStyle) {
+  let { style } = inlineSizeStyle.cssRules[0];
 
   style.maxBlockSize = 'initial';
   style.insetBlockStart = 'initial';
@@ -518,31 +523,8 @@ function reposition(reference, popover) {
   }
 }
 
-/**
- * Get a CSS rule with a selector in an element containing <style> tags.
- * @param  {Element|ShadowRoot} styleParent
- * @param  {string} selectorText
- * @return {CSSStyleRule}
-  */
-function getCSSRule(styleParent, selectorText) {
-  let style;
-  for (style of styleParent.querySelectorAll('style')) {
-
-    // Catch this error. e.g. browser extension adds style tags.
-    //   Uncaught DOMException: CSSStyleSheet.cssRules getter:
-    //   Not allowed to access cross-origin stylesheet
-    let cssRules;
-    try { cssRules = style.sheet?.cssRules; } catch { continue; }
-
-    for (let rule of cssRules ?? [])
-      if (rule.selectorText === selectorText) return rule;
-  }
-
-  return {};
-}
-
 if (!globalThis.customElements.get('x-selectlist')) {
   globalThis.customElements.define('x-selectlist', SelectListElement);
 }
 
-export default SelectListElement;
+export default SelectListElement;
\ No newline at end of file