Merge a129236 into 064251c

Azure · Oct 9, 2018 · c3e7977 · c3e7977
2 parents 064251c + a129236
commit c3e7977
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/client/src/app/site/spec-picker/spec-feature-list/spec-feature-list.component.html b/client/src/app/site/spec-picker/spec-feature-list/spec-feature-list.component.html
@@ -5,13 +5,15 @@ <h2>{{title}}</h2>
 
     <div [load-image]="feature.iconUrl" class="icon-medium"></div>
     <div class="feature-body">
-      <h4>{{feature.title}}</h4>
-      <div class="feature-description" [id]="feature.title | removeSpaces">{{feature.description}}
-        <a *ngIf="feature.learnMoreUrl"
-          [href]="feature.learnMoreUrl"
-          target="_blank"
-          [attr.aria-label]="feature.title"
-          [attr.aria-describedby]="feature.title | removeSpaces">{{'topBar_learnMore' | translate}}</a>
+      <h4 [id]="feature.title | removeSpaces">{{feature.title}}</h4>
+      <div class="feature-description">{{feature.description}}
+        <div *ngIf="feature.learnMoreUrl"
+          [attr.aria-labelledBy]="feature.title | removeSpaces"
+          [attr.aria-label]="'topBar_learnMore' | translate">
+          <a [href]="feature.learnMoreUrl"
+            target="_blank"
+            [attr.aria-describedby]="feature.title | removeSpaces">{{'topBar_learnMore' | translate}}</a>
+        </div>
       </div>
     </div>
   </div>

diff --git a/client/src/app/site/spec-picker/spec-picker.component.html b/client/src/app/site/spec-picker/spec-picker.component.html
@@ -52,10 +52,10 @@ <h2 class="tiers-header" *ngIf="specManager.selectedSpecGroup.recommendedSpecs.l
         <div class="spec-expander" *ngIf="showExpander">
           <span (click)="specManager.selectedSpecGroup.isExpanded = !specManager.selectedSpecGroup.isExpanded"
                 (keydown)="onExpandKeyPress($event)"
-                role="link"
+                role="button"
                 tabindex="0"
                 [attr.aria-expanded]="specManager.selectedSpecGroup.isExpanded"
-                [attr.aria-controls]="specManager.selectedSpecGroup.id + specManager.selectedSpecGroup.selectedSpec">
+                [attr.aria-controls]="specManager.selectedSpecGroup.id + specManager.selectedSpecGroup.selectedSpec.skuCode">
 
             <span [load-image]="!specManager.selectedSpecGroup.isExpanded ? 'image/caret-down.svg' : 'image/caret-up.svg'" class="expand-icon"></span>
             <a>{{ (!specManager.selectedSpecGroup.isExpanded ? 'seeAllOptions' : 'seeRecommendedOptions') | translate}}</a>