chore: remove erroneous focusable class, remove space from hidden men…

…u group

packages/menu/src/Menu.ts

@@ -15,6 +15,7 @@ import {
     html,
     PropertyValues,
     SizedMixin,
+    SpectrumElement,
     TemplateResult,
 } from '@spectrum-web-components/base';
 import {
@@ -28,7 +29,6 @@ import type {
     MenuItemRemovedEvent,
 } from './MenuItem.js';
 import menuStyles from './menu.css.js';
-import { Focusable } from '@spectrum-web-components/shared';
 
 export interface MenuChildItem {
     menuItem: MenuItem;
@@ -61,7 +61,7 @@ function elementIsOrContains(
  *   When the `selects` attribute is not present a `value` will not be maintained and the Menu
  *   Item children of this Menu will not have their `selected` state managed.
  */
-export class Menu extends SizedMixin(Focusable) {
+export class Menu extends SizedMixin(SpectrumElement) {
     public static override get styles(): CSSResultArray {
         return [menuStyles];
     }
@@ -96,10 +96,6 @@ export class Menu extends SizedMixin(Focusable) {
     @query('slot:not([name])')
     public menuSlot!: HTMLSlotElement;
 
-    public override get focusElement(): HTMLElement {
-        return this.menuSlot;
-    }
-
     private childItemSet = new Set<MenuItem>();
     public focusedItemIndex = 0;
     public focusInItemIndex = 0;

packages/menu/src/MenuItem.ts

@@ -200,7 +200,7 @@ export class MenuItem extends LikeAnchor(
     private anchorElement!: HTMLAnchorElement;
 
     public override get focusElement(): HTMLElement {
-        return this.anchorElement || this;
+        return this;
     }
 
     protected get hasIcon(): boolean {

packages/menu/src/menu-group.css

@@ -19,34 +19,6 @@ governing permissions and limitations under the License.
     flex-direction: column;
 }
 
-/**
- * The following corrects for positioning of the header when changing the
- * direction of the content while https://github.com/adobe/spectrum-css/issues/794
- * persists at the Spectrum CSS level.
- */
-
-/* :host([dir='ltr']) .header {
-    padding: 0 var(--spectrum-global-dimension-size-450) 0
-        var(--spectrum-global-dimension-size-150);
-}
-
-:host([dir='rtl']) .header {
-    padding: 0 var(--spectrum-global-dimension-size-150) 0
-        var(--spectrum-global-dimension-size-450);
-}
-
-sp-menu {
-    --swc-menu-width: 100%;
-}
-
-:host(:last-child) sp-menu {
-    margin-bottom: 0;
-}
-
-:host(:first-child) .header[hidden] + sp-menu {
-    margin-top: 0;
-}
-
 [hidden] {
     display: none !important;
-} */
+}

packages/menu/src/spectrum-config.js

@@ -311,7 +311,6 @@ const config = {
             fileName: 'menu',
             excludeByComponents: [
                 builder.class('spectrum-Menu-divider'),
-                // builder.class('spectrum-Menu-sectionHeading'),
                 builder.class('spectrum-menu-itemSelection'),
                 {
                     type: 'class',