address PR feedback

Author: radium-v

packages/web-components/src/accordion-item/define-async.ts

@@ -1,5 +1,5 @@
 import { RenderableFASTElement } from '@microsoft/fast-html';
-import { AccordionItem } from './accordion-item.js';
 import { definition } from './accordion-item.definition-async.js';
+import { AccordionItem } from './accordion-item.js';
 
 RenderableFASTElement(AccordionItem).defineAsync(definition);

packages/web-components/src/accordion/accordion.definition-async.ts

@@ -1,6 +1,14 @@
 import { type PartialFASTElementDefinition } from '@microsoft/fast-element';
 import { tagName } from './accordion.options.js';
 
+/**
+ * The async definition configuration for the fluent-accordion element.
+ *
+ * @public
+ * @remarks
+ * This is used in server-side rendering (SSR) scenarios where the template
+ * is provided as a deferred option to be hydrated later.
+ */
 export const declarativeDefinition: PartialFASTElementDefinition = {
   name: tagName,
   templateOptions: 'defer-and-hydrate',

packages/web-components/src/dropdown/dropdown.base.ts

@@ -446,9 +446,7 @@ export class BaseDropdown extends FASTElement {
   public get enabledOptions(): DropdownOption[] {
     return (
       this.listbox?.enabledOptions ??
-      Array.from(this.querySelectorAll('*')).filter<DropdownOption>(
-        (o): o is DropdownOption => isDropdownOption(o) && !o.disabled,
-      )
+      Array.from(this.querySelectorAll('*')).filter((o): o is DropdownOption => isDropdownOption(o) && !o.disabled)
     );
   }
 

packages/web-components/src/menu/define-async.ts

@@ -1,5 +1,5 @@
 import { RenderableFASTElement } from '@microsoft/fast-html';
-import { Menu } from './menu.js';
 import { definition } from './menu.definition-async.js';
+import { Menu } from './menu.js';
 
 RenderableFASTElement(Menu).defineAsync(definition);

packages/web-components/src/textarea/textarea.spec.ts

@@ -151,18 +151,15 @@ test.describe('TextArea', () => {
     }) => {
       const { element } = fastPage;
 
-      await page.addInitScript({
-        content: `
-          const originalSupports = CSS.supports.bind(CSS);
-          window.__originalCSSSupports = originalSupports;
-          CSS.supports = (property, value) => {
-            if (property === 'field-sizing: content' || (property === 'field-sizing' && value === 'content')) {
-              console.log('Mocking CSS.supports to return false for', property);
-              return false;
-            }
-            return originalSupports(property, value);
-          };
-        `,
+      await page.addInitScript(() => {
+        const originalSupports = CSS.supports.bind(CSS);
+        (window as any).__originalCSSSupports = originalSupports;
+        (CSS as any).supports = (property: string, value: string) => {
+          if (property === 'field-sizing: content' || (property === 'field-sizing' && value === 'content')) {
+            return false;
+          }
+          return originalSupports(property, value);
+        };
       });
 
       // Ensures the next navigation creates a new context where the patched CSS.supports is in effect.

packages/web-components/src/tree-item/tree-item.base.ts

@@ -1,4 +1,4 @@
-import { attr, css, type ElementStyles, FASTElement, observable, Updates } from '@microsoft/fast-element';
+import { attr, css, type ElementStyles, FASTElement, observable } from '@microsoft/fast-element';
 import { toggleState } from '../utils/element-internals.js';
 import { isTreeItem } from './tree-item.options.js';