style(radio): refactor tabindex code

elements/pf-radio/pf-radio.ts

@@ -1,7 +1,10 @@
 import { LitElement, html, type TemplateResult } from 'lit';
 import { customElement } from 'lit/decorators/custom-element.js';
-import styles from './pf-radio.css';
 import { property } from 'lit/decorators/property.js';
+import { observes } from '@patternfly/pfe-core/decorators/observes.js';
+import { state } from 'lit/decorators/state.js';
+
+import styles from './pf-radio.css';
 
 export class PfRadioChangeEvent extends Event {
   constructor(public event: Event, public value: string) {
@@ -16,46 +19,89 @@ export class PfRadioChangeEvent extends Event {
 @customElement('pf-radio')
 export class PfRadio extends LitElement {
   static readonly styles: CSSStyleSheet[] = [styles];
+
   static formAssociated = true;
+
   static shadowRootOptions: ShadowRootInit = {
     ...LitElement.shadowRootOptions,
     delegatesFocus: true,
   };
 
-  @property({
-    type: Boolean,
-    attribute: 'checked',
-    converter: {
-      fromAttribute: value => value === 'true',
-    },
-    reflect: true,
-  })
+  @property({ type: Boolean, reflect: true })
   checked = false;
 
-  @property({
-    type: Boolean,
-    attribute: 'disabled',
-    converter: {
-      fromAttribute: value => value === 'true',
-    },
-    reflect: true,
-  })
+  @property({ type: Boolean, reflect: true })
   disabled = false;
 
-  @property({ attribute: 'name', reflect: true }) name = '';
-  @property({ attribute: 'label', reflect: true }) label?: string;
-  @property({ attribute: 'value', reflect: true }) value = '';
-  @property({ attribute: 'id', reflect: true }) id = '';
-  @property({ attribute: 'tabindex', reflect: true }) tabIndex = -1;
+  @property({ reflect: true }) name = '';
+
+  @property({ reflect: true }) label?: string;
+
+  @property({ reflect: true }) value = '';
+
+  @state() private focusable = false;
 
-  constructor() {
-    super();
+  /** Radio groups: instances.get(groupName).forEach(pfRadio => { ... }) */
+  private static instances = new Map<string, Set<PfRadio>>();
+
+  private static selected = new Map<string, PfRadio>;
+
+  static {
+    globalThis.addEventListener('keydown', e => {
+      switch (e.key) {
+        case 'Tab':
+          this.instances.forEach((radioSet, groupName) => {
+            const selected = this.selected.get(groupName);
+            [...radioSet].forEach((radio, i, radios) => {
+              // the radio group has a selected element
+              // it should be the only focusable member of the group
+              if (selected) {
+                radio.focusable = radio === selected;
+              // when Shift-tabbing into a group, only the last member should be selected
+              } else if (e.shiftKey) {
+                radio.focusable = radio === radios.at(-1);
+              // otherwise, the first member must be focusable
+              } else {
+                radio.focusable = i === 0;
+              }
+            });
+          });
+          break;
+      }
+    });
   }
 
   connectedCallback(): void {
     super.connectedCallback();
     this.addEventListener('keydown', this.#onKeydown);
-    document.addEventListener('keydown', this.#onKeyPress);
+  }
+
+  @observes('checked')
+  protected checkedChanged(): void {
+    if (this.checked) {
+      PfRadio.selected.set(this.name, this);
+    }
+  }
+
+  @observes('name')
+  protected nameChanged(oldName: string): void {
+    // reset the map of groupname to selected radio button
+    if (PfRadio.selected.get(oldName) === this) {
+      PfRadio.selected.delete(oldName);
+      PfRadio.selected.set(this.name, this);
+    }
+    if (typeof oldName === 'string') {
+      PfRadio.instances.get(oldName)?.delete(this);
+    }
+    if (!PfRadio.instances.has(this.name)) {
+      PfRadio.instances.set(this.name, new Set());
+    }
+    PfRadio.instances.get(this.name)?.add(this);
+  }
+
+  disconnectedCallback(): void {
+    PfRadio.instances.get(this.name)?.delete(this);
+    super.disconnectedCallback();
   }
 
   #onRadioButtonClick(event: Event) {
@@ -66,44 +112,17 @@ export class PfRadio extends LitElement {
         radioGroup = root.querySelectorAll('pf-radio');
         radioGroup.forEach((radio: PfRadio) => {
           const element: HTMLElement = radio as HTMLElement;
+          // avoid removeAttribute: set checked property instead
+          // even better: listen for `change` on the shadow input,
+          // and recalculate state from there.
           element?.removeAttribute('checked');
-          element.tabIndex = -1;
         });
         this.checked = true;
-        this.tabIndex = 0;
         this.dispatchEvent(new PfRadioChangeEvent(event, this.value));
       }
     }
   }
 
-  // Function to handle tab key navigation
-  #onKeyPress = (event: KeyboardEvent) => {
-    const root: Node = this.getRootNode();
-    if (root instanceof Document || root instanceof ShadowRoot) {
-      const radioGroup: NodeListOf<PfRadio> = root.querySelectorAll('pf-radio');
-      const isRadioChecked: boolean = Array.from(radioGroup).some(
-        (radio: PfRadio) => radio.checked
-      );
-      if (event.key === 'Tab') {
-        radioGroup.forEach((radio: PfRadio) => {
-          radio.tabIndex = radio.checked ? 0 : -1;
-        });
-        if (!isRadioChecked) {
-          radioGroup.forEach((radio: PfRadio, index: number) => {
-            radio.tabIndex = -1;
-            if (event.shiftKey) {
-              if (index === radioGroup.length - 1) {
-                radio.tabIndex = 0;
-              }
-            } else if (index === 0) {
-              radio.tabIndex = 0;
-            }
-          });
-        }
-      }
-    }
-  };
-
   // Function to handle keyboard navigation
   #onKeydown = (event: KeyboardEvent) => {
     const arrowKeys: string[] = ['ArrowDown', 'ArrowRight', 'ArrowUp', 'ArrowLeft'];
@@ -113,7 +132,6 @@ export class PfRadio extends LitElement {
         const radioGroup: NodeListOf<PfRadio> = root.querySelectorAll('pf-radio');
         radioGroup.forEach((radio: PfRadio, index: number) => {
           this.checked = false;
-          this.tabIndex = 0;
 
           if (radio === event.target) {
             const isArrowDownOrRight: boolean = ['ArrowDown', 'ArrowRight'].includes(event.key);
@@ -125,6 +143,9 @@ export class PfRadio extends LitElement {
             const nextIndex: number = (index + direction + radioGroup.length) % radioGroup.length;
             radioGroup[nextIndex].focus();
             radioGroup[nextIndex].checked = true;
+            // TODO: move this to an @observes
+            // consider the api of this event.
+            // do we add the group to it? do we fire from every element on every change?
             this.dispatchEvent(new PfRadioChangeEvent(event, radioGroup[nextIndex].value));
           }
         });
@@ -135,15 +156,15 @@ export class PfRadio extends LitElement {
   render(): TemplateResult<1> {
     return html`
       <input
+        id="radio"
+        type="radio"
         @click=${this.#onRadioButtonClick}
-        id=${this.id}
         .name=${this.name}
-        type='radio'
         value=${this.value}
-        tabindex=${this.tabIndex}
+        tabindex=${this.focusable ? 0 : -1}
         .checked=${this.checked}
-      />
-      <label for=${this.id}>${this.label}</label>
+      >
+      <label for="radio">${this.label}</label>
     `;
   }
 }