Merge pull request #2261 from crazyserver/MOBILE-3301

MOBILE-3301 ux: Add aria-label to context more buttons

Author: jleyva

scripts/langindex.json

@@ -1476,6 +1476,7 @@
   "core.digitalminor_desc": "moodle",
   "core.discard": "local_moodlemobileapp",
   "core.dismiss": "local_moodlemobileapp",
+  "core.displayoptions": "atto_media",
   "core.done": "survey",
   "core.download": "moodle",
   "core.downloaded": "local_moodlemobileapp",

src/addon/mod/forum/components/index/addon-mod-forum-index.html

@@ -67,7 +67,7 @@ <h2 text-wrap>
                                 <core-icon name="fa-star" class="addon-forum-star" *ngIf="!discussion.pinned && discussion.starred"></core-icon>
                                 <core-format-text [text]="discussion.subject" contextLevel="module" [contextInstanceId]="module.id" [courseId]="courseId"></core-format-text>
                             </h2>
-                            <button ion-button icon-only clear color="dark" (click)="showOptionsMenu($event, discussion)" *ngIf="canPin || discussion.canlock || discussion.canfavourite">
+                            <button ion-button icon-only clear color="dark" (click)="showOptionsMenu($event, discussion)" *ngIf="canPin || discussion.canlock || discussion.canfavourite" [attr.aria-label]="('core.displayoptions' | translate)">
                                 <core-icon name="more"></core-icon>
                             </button>
                         </div>

src/addon/mod/forum/components/post/addon-mod-forum-post.html

@@ -10,7 +10,7 @@ <h2 text-wrap>
                 <ion-note float-end padding-left text-end *ngIf="trackPosts && !post.postread" [attr.aria-label]="'addon.mod_forum.unread' | translate">
                     <core-icon name="fa-circle" color="primary"></core-icon>
                 </ion-note>
-                <button ion-button icon-only clear color="dark" (click)="showOptionsMenu($event)" *ngIf="optionsMenuEnabled">
+                <button ion-button icon-only clear color="dark" (click)="showOptionsMenu($event)" *ngIf="optionsMenuEnabled" [attr.aria-label]="('core.displayoptions' | translate)">
                     <core-icon name="more"></core-icon>
                 </button>
             </div>
@@ -26,7 +26,7 @@ <h3 *ngIf="post.userfullname">{{post.userfullname}}</h3>
                     <ion-note float-end padding-left text-end *ngIf="trackPosts && !post.postread" [attr.aria-label]="'addon.mod_forum.unread' | translate">
                         <core-icon name="fa-circle" color="primary"></core-icon>
                     </ion-note>
-                    <button ion-button icon-only clear color="dark" (click)="showOptionsMenu($event)" *ngIf="optionsMenuEnabled">
+                    <button ion-button icon-only clear color="dark" (click)="showOptionsMenu($event)" *ngIf="optionsMenuEnabled" [attr.aria-label]="('core.displayoptions' | translate)">
                         <core-icon name="more"></core-icon>
                     </button>
                 </ng-container>

src/app/app.scss

@@ -683,6 +683,7 @@ ion-app.app-root {
 
   .alert-message {
       overflow-y: auto;
+      user-select: text;
   }
   ion-alert.core-nohead {
 

src/assets/lang/en.json

@@ -1476,6 +1476,7 @@
     "core.digitalminor_desc": "Please ask your parent/guardian to contact:",
     "core.discard": "Discard",
     "core.dismiss": "Dismiss",
+    "core.displayoptions": "Display options",
     "core.done": "Done",
     "core.download": "Download",
     "core.downloaded": "Downloaded",

src/components/context-menu/context-menu.ts

@@ -70,7 +70,7 @@ export class CoreContextMenuComponent implements OnInit, OnDestroy {
      */
     ngOnInit(): void {
         this.icon = this.icon || 'more';
-        this.ariaLabel = this.ariaLabel || this.title || this.translate.instant('core.info');
+        this.ariaLabel = this.ariaLabel || this.title || this.translate.instant('core.displayoptions');
     }
 
     /**

src/core/courses/components/course-progress/core-courses-course-progress.html

@@ -33,7 +33,7 @@ <h2>
              <ion-icon *ngIf="downloadCourseEnabled && prefetchCourseData.downloadSucceeded && !showSpinner" class="core-icon-downloaded" name="cloud-done" color="success" [attr.aria-label]="'core.downloaded' | translate" role="status"></ion-icon>
 
             <!-- Options menu. -->
-            <button ion-button icon-only clear color="dark" (click)="showCourseOptionsMenu($event)" *ngIf="!showSpinner">
+            <button ion-button icon-only clear color="dark" (click)="showCourseOptionsMenu($event)" *ngIf="!showSpinner" [attr.aria-label]="('core.displayoptions' | translate)">
                 <core-icon name="more"></core-icon>
             </button>
         </div>

src/lang/en.json

@@ -74,6 +74,7 @@
     "digitalminor_desc": "Please ask your parent/guardian to contact:",
     "discard": "Discard",
     "dismiss": "Dismiss",
+    "displayoptions": "Display options",
     "done": "Done",
     "download": "Download",
     "downloaded": "Downloaded",