mirror of
https://github.com/material-components/material-web.git
synced 2024-10-26 21:56:56 +03:00
bd88880f78
Ensure pressing escape while on a closed submenu anchor, closes the parent menu Previously we were not letting a user close a menu when they were focused on any item with keepOpen = true. This included submenu anchors. This change adds an escape (hehe) hatch to ensure that even on items with keepOpen = true, escape closes the menu. This is what most users expect and is what is the standard on menus around the web. PiperOrigin-RevId: 587874713
119 lines
2.8 KiB
TypeScript
119 lines
2.8 KiB
TypeScript
/**
|
|
* @license
|
|
* Copyright 2022 Google LLC
|
|
* SPDX-License-Identifier: Apache-2.0
|
|
*/
|
|
|
|
// import 'jasmine'; (google3-only)
|
|
import './menu.js';
|
|
import './sub-menu.js';
|
|
|
|
import {html, render} from 'lit';
|
|
|
|
import {createTokenTests} from '../testing/tokens.js';
|
|
|
|
import {MenuItemHarness} from './harness.js';
|
|
import {MdMenu} from './menu.js';
|
|
import {MdMenuItem} from './menu-item.js';
|
|
|
|
describe('<md-menu>', () => {
|
|
describe('.styles', () => {
|
|
createTokenTests(MdMenu.styles);
|
|
});
|
|
|
|
let root: HTMLDivElement;
|
|
|
|
beforeEach(() => {
|
|
root = document.createElement('div');
|
|
document.body.appendChild(root);
|
|
});
|
|
|
|
afterEach(() => {
|
|
root?.remove();
|
|
});
|
|
|
|
it('escape on list root closes menu', async () => {
|
|
render(
|
|
html`
|
|
<button>OpenMenu</button>
|
|
<md-menu quick></md-menu>
|
|
`,
|
|
root,
|
|
);
|
|
|
|
const button = root.querySelector('button')!;
|
|
const menu = root.querySelector('md-menu')!;
|
|
menu.anchorElement = button;
|
|
menu.show();
|
|
|
|
expect(menu.open).toBeTrue();
|
|
|
|
const escapeKeydownEvent = new KeyboardEvent('keydown', {
|
|
key: 'Escape',
|
|
code: 'Escape',
|
|
bubbles: true,
|
|
composed: true,
|
|
cancelable: true,
|
|
});
|
|
menu.dispatchEvent(escapeKeydownEvent);
|
|
|
|
await menu.updateComplete;
|
|
|
|
expect(menu.open).toBeFalse();
|
|
expect(escapeKeydownEvent.defaultPrevented).toBeTrue();
|
|
});
|
|
|
|
// Regression test for b/314706578.
|
|
it('escape on submenu items closes menu', async () => {
|
|
render(
|
|
html`
|
|
<button>OpenMenu</button>
|
|
<md-menu quick>
|
|
<md-sub-menu hover-open-delay="0" hover-close-delay="0">
|
|
<md-menu-item id="submenu-item" slot="item">
|
|
<div slot="headline">Link Item 1</div>
|
|
</md-menu-item>
|
|
<md-menu quick slot="menu">
|
|
<md-menu-item slot="item">
|
|
<div slot="headline">Submenu Item 1</div>
|
|
</md-menu-item>
|
|
</md-menu>
|
|
</md-sub-menu>
|
|
</md-menu>
|
|
`,
|
|
root,
|
|
);
|
|
|
|
const button = root.querySelector('button')!;
|
|
const menu = root.querySelector('md-menu')!;
|
|
const submenuItemHarness = new MenuItemHarness(
|
|
menu.querySelector('#submenu-item')!,
|
|
);
|
|
menu.anchorElement = button;
|
|
menu.show();
|
|
|
|
expect(menu.open).toBeTrue();
|
|
|
|
const escapeKeydownEvent = new KeyboardEvent('keydown', {
|
|
key: 'Escape',
|
|
code: 'Escape',
|
|
bubbles: true,
|
|
composed: true,
|
|
cancelable: true,
|
|
});
|
|
const interactiveElement = await submenuItemHarness.getInteractiveElement();
|
|
interactiveElement.dispatchEvent(escapeKeydownEvent);
|
|
|
|
await menu.updateComplete;
|
|
|
|
expect(menu.open).toBeFalse();
|
|
expect(escapeKeydownEvent.defaultPrevented).toBeTrue();
|
|
});
|
|
});
|
|
|
|
describe('<md-menu-item>', () => {
|
|
describe('.styles', () => {
|
|
createTokenTests(MdMenuItem.styles);
|
|
});
|
|
});
|