Skip to content

Commit a7135c9

Browse files
committed
fixup! fix(cdk/menu): update docs to reflect current implementation and add correct role for triggers
1 parent 76185ff commit a7135c9

File tree

3 files changed

+14
-26
lines changed

3 files changed

+14
-26
lines changed

src/cdk/menu/menu-item-checkbox.ts

+1
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import {CdkMenuItem} from './menu-item';
1919
exportAs: 'cdkMenuItemCheckbox',
2020
host: {
2121
'role': 'menuitemcheckbox',
22+
'[class.cdk-menu-item-checkbox]': 'true',
2223
},
2324
providers: [
2425
{provide: CdkMenuItemSelectable, useExisting: CdkMenuItemCheckbox},

src/cdk/menu/menu-item-radio.ts

+1
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ let nextId = 0;
2929
exportAs: 'cdkMenuItemRadio',
3030
host: {
3131
'role': 'menuitemradio',
32+
'[class.cdk-menu-item-radio]': 'true',
3233
},
3334
providers: [
3435
{provide: CdkMenuItemSelectable, useExisting: CdkMenuItemRadio},

src/cdk/menu/menu.md

+12-26
Original file line numberDiff line numberDiff line change
@@ -26,32 +26,18 @@ make any assumptions about how elements should be styled. You are expected to ap
2626
CSS styles, but the directives do apply CSS classes to make it easier for you to add custom styles.
2727
The available CSS classes are listed below, by directive.
2828

29-
#### `cdkMenu`
30-
31-
`cdk-menu`
32-
: Always applied
33-
`cdk-menu-inline`
34-
: Applied if the menu is an [inline menu](#menu-content)
35-
36-
#### `cdkMenuBar`
37-
38-
`cdk-menu-bar`
39-
: Always applied
40-
41-
#### `cdkMenuGroup`
42-
43-
`cdk-menu-group`
44-
: Always applied
45-
46-
#### `cdkMenuItem`
47-
48-
`cdk-menu-item`
49-
: Always applied
50-
51-
#### `cdkMenuTriggerFor`
52-
53-
`cdk-menu-trigger`
54-
: Always applied
29+
| Directive | CSS Class | Applied... |
30+
|:----------------------|--------------------------|------------------------------------------------|
31+
| `cdkMenu` | `cdk-menu` | Always |
32+
| `cdkMenu` | `cdk-menu-inline` | If the menu is an [inline menu](#menu-content) |
33+
| `cdkMenuBar` | `cdk-menu-bar` | Always |
34+
| `cdkMenuGroup` | `cdk-menu-group` | Always |
35+
| `cdkMenuItem` | `cdk-menu-item` | Always |
36+
| `cdkMenuItemCheckbox` | `cdk-menu-item` | Always |
37+
| `cdkMenuItemCheckbox` | `cdk-menu-item-checkbox` | Always |
38+
| `cdkMenuItemRadio` | `cdk-menu-item` | Always |
39+
| `cdkMenuItemRadio` | `cdk-menu-item-radio` | Always |
40+
| `cdkMenuTriggerFor` | `cdk-menu-trigger` | Always |
5541

5642
### Getting started
5743

0 commit comments

Comments
 (0)