refactor(list)!: refactor list to reuse ListController

fixes #4967

also fixes list-root focus ring and focusout

BREAKING CHANGE: the new ListController behavior no longer waits for event.preventDefault asynchronously because it was causing keyboard navigations to scroll the page.

PiperOrigin-RevId: 568300155

menu/internal/list-controller.ts → list/internal/list-controller.ts

@@ -206,23 +206,23 @@ export class ListController<Item extends ListItem> {
   /**
    * Listener to be bound to the `deactivate-items` item event.
    */
-  onDeactivateItems() {
+  onDeactivateItems = () => {
     const items = this.items;
 
     for (const item of items) {
       this.deactivateItem(item);
     }
-  }
+  };
 
   /**
    * Listener to be bound to the `request-activation` item event..
    */
-  onRequestActivation(event: Event) {
+  onRequestActivation = (event: Event) => {
     this.onDeactivateItems();
     const target = event.target as Item;
     this.activateItem(target);
     target.focus();
-  }
+  };
 
   /**
    * Listener to be bound to the `slotchange` event for the slot that renders