Skip to content

Commit 6e89905

Browse files
committed
Remove redundant element roles.
1 parent 4f078ec commit 6e89905

File tree

2 files changed

+2
-18
lines changed

2 files changed

+2
-18
lines changed

src/lib/dialog/dialog-content-directives.ts

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,7 @@ export class MdDialogClose {
2323
* Title of a dialog element. Stays fixed to the top of the dialog when scrolling.
2424
*/
2525
@Directive({
26-
selector: '[md-dialog-title], [mat-dialog-title]',
27-
host: {
28-
role: 'heading'
29-
}
26+
selector: '[md-dialog-title], [mat-dialog-title]'
3027
})
3128
export class MdDialogTitle { }
3229

@@ -35,10 +32,7 @@ export class MdDialogTitle { }
3532
* Scrollable content container of a dialog.
3633
*/
3734
@Directive({
38-
selector: '[md-dialog-content], md-dialog-content, [mat-dialog-content], mat-dialog-content',
39-
host: {
40-
role: 'main'
41-
}
35+
selector: '[md-dialog-content], md-dialog-content, [mat-dialog-content], mat-dialog-content'
4236
})
4337
export class MdDialogContent { }
4438

src/lib/dialog/dialog.spec.ts

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -343,16 +343,6 @@ describe('MdDialog', () => {
343343
expect(button.getAttribute('aria-label')).toBe('Best close button ever');
344344
});
345345

346-
it('should add a role to the dialog title', () => {
347-
let header = overlayContainerElement.querySelector('[md-dialog-title]');
348-
expect(header.getAttribute('role')).toBe('heading');
349-
});
350-
351-
it('should add a role to the dialog content', () => {
352-
let content = overlayContainerElement.querySelector('md-dialog-content');
353-
expect(content.getAttribute('role')).toBe('main');
354-
});
355-
356346
});
357347
});
358348

0 commit comments

Comments
 (0)