Skip to content

Commit 6c124f0

Browse files
[mgt-agenda][accessibility] removing non-actionable content from tabindex (#1645)
Co-authored-by: Sébastien Levert <[email protected]>
1 parent 30ec91b commit 6c124f0

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

packages/mgt-components/src/components/mgt-agenda/mgt-agenda.ts

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -345,9 +345,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
345345
return html`
346346
<div class="event">
347347
<div class="event-time-container">
348-
<div tabindex=0 class="event-time" aria-label="${this.getEventTimeString(event)}">${this.getEventTimeString(
349-
event
350-
)}</div>
348+
<div class="event-time" aria-label="${this.getEventTimeString(event)}">${this.getEventTimeString(event)}</div>
351349
</div>
352350
<div class="event-details-container">
353351
${this.renderTitle(event)} ${this.renderLocation(event)} ${this.renderAttendees(event)}
@@ -371,7 +369,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
371369
return (
372370
this.renderTemplate('header', { header }, 'header-' + header) ||
373371
html`
374-
<div tabindex=0 class="header" aria-label="${header}">${header}</div>
372+
<div class="header" aria-label="${header}">${header}</div>
375373
`
376374
);
377375
}
@@ -386,7 +384,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
386384
*/
387385
protected renderTitle(event: MicrosoftGraph.Event): TemplateResult {
388386
return html`
389-
<div tabindex=0 aria-label=${event.subject} class="event-subject">${event.subject}</div>
387+
<div aria-label=${event.subject} class="event-subject">${event.subject}</div>
390388
`;
391389
}
392390

@@ -406,9 +404,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
406404
return html`
407405
<div class="event-location-container">
408406
<div class="event-location-icon">${getSvg(SvgIcon.OfficeLocation)}</div>
409-
<div tabindex=0 class="event-location" aria-label="${event.location.displayName}">${
410-
event.location.displayName
411-
}</div>
407+
<div class="event-location" aria-label="${event.location.displayName}">${event.location.displayName}</div>
412408
</div>
413409
`;
414410
}

0 commit comments

Comments
 (0)