Skip to content

Commit 860b642

Browse files
committed
fixup! fix(cdk-experimental/menu): move shared menu logic to base class
1 parent 8e2790e commit 860b642

File tree

2 files changed

+1
-13
lines changed

2 files changed

+1
-13
lines changed

src/material-experimental/menubar/menubar.spec.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,15 +39,6 @@ describe('MatMenuBar', () => {
3939
expect(nativeMatMenubar.getAttribute('tabindex')).toBe('0');
4040
});
4141

42-
it('should toggle aria-orientation attribute', () => {
43-
expect(nativeMatMenubar.getAttribute('aria-orientation')).toBe('horizontal');
44-
45-
matMenubar.orientation = 'vertical';
46-
fixture.detectChanges();
47-
48-
expect(nativeMatMenubar.getAttribute('aria-orientation')).toBe('vertical');
49-
});
50-
5142
it('should toggle focused items on left/right click', () => {
5243
nativeMatMenubar.focus();
5344

src/material-experimental/menubar/menubar.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,7 @@ import {
2727
encapsulation: ViewEncapsulation.None,
2828
changeDetection: ChangeDetectionStrategy.OnPush,
2929
host: {
30-
'role': 'menubar',
31-
'class': 'cdk-menu-bar mat-menubar',
32-
'tabindex': '0',
33-
'[attr.aria-orientation]': 'orientation',
30+
'[class.mat-menubar]': 'true',
3431
},
3532
providers: [
3633
{provide: CdkMenuGroup, useExisting: MatMenuBar},

0 commit comments

Comments
 (0)