[ACA-2582] menu menuitem roles (#1315)

* chore: wip

* chore: wip mark will poke around

* chore: removed unused directive

* remove useless directive

* cleanup code

Co-authored-by: Denys Vuika <denys.vuika@gmail.com>

src/app/components/create-menu/create-menu.component.html

@@ -37,7 +37,7 @@
   [overlapTrigger]="false"
   yPosition="below"
 >
-  <ng-container *ngFor="let action of createActions; trackBy: trackById">
+  <div role="menu" *ngFor="let action of createActions; trackBy: trackById">
     <app-toolbar-menu-item [actionRef]="action"></app-toolbar-menu-item>
-  </ng-container>
+  </div>
 </mat-menu>

src/app/components/toolbar/toolbar-menu-item/toolbar-menu-item.component.html

@@ -2,6 +2,7 @@
   <ng-container *ngSwitchCase="'menu'">
     <button
       mat-menu-item
+      role="menuitem"
       [disabled]="actionRef.disabled"
       [matMenuTriggerFor]="childMenu"
     >
@@ -29,6 +30,7 @@
   <ng-container *ngSwitchDefault>
     <button
       [id]="actionRef.id"
+      role="button"
       mat-menu-item
       [role]="'button'"
       color="primary"