fix/ remove iframe tabs

RoyalBoroughKingston · May 9, 2019 · 6bd8943 · 6bd8943
1 parent 69adff8
commit 6bd8943
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 4 deletions.
diff --git a/src/partials/image-viewer.html b/src/partials/image-viewer.html
@@ -26,7 +26,7 @@
       </div>
     </div>
   </div>
-  <a class="image-viewer__text"
+  <a tabindex=0 class="image-viewer__text"
     ><i aria-hidden="true" class="image-viewer__icon fas fa-camera"></i> View images (4)</a
   >
 </div>
diff --git a/src/partials/page/media.html b/src/partials/page/media.html
@@ -32,7 +32,7 @@ <h5 class="section__subtitle">Image</h5>
               />
             </div>
           </div>
-          <a class="image-viewer__text"
+          <a tabindex=0 class="image-viewer__text"
             ><i aria-hidden="true" class="image-viewer__icon fas fa-camera"></i> View images (4)</a
           >
         </div>

diff --git a/src/scss/utils/_typography.scss b/src/scss/utils/_typography.scss
@@ -81,6 +81,11 @@ blockquote {
 /* Remove underline */
 a {
   text-decoration: none;
+
+  &:focus,
+  &:active {
+    text-decoration: underline;
+  }
 }
 
 /* Remove italics */

diff --git a/src/service.html b/src/service.html
@@ -36,6 +36,7 @@ <h2 class="service__title">What is this group?</h2>
             <include src="./image-viewer.html"></include>
             <!--Mobile Embeded video-->
             <iframe
+              tabindex="-1"
               title="Service video"
               class="service__video video video--mobile"
               src="https://www.youtube.com/embed/tgbNymZ7vqY"
@@ -71,6 +72,7 @@ <h2 class="service__title service__title--accordion">
           <aside class="col-xsmall-12 col-medium-4">
             <!--Dekstop Embeded video-->
             <iframe
+              tabindex="-1"
               title="Service video"
               class="service__video video"
               src="https://www.youtube.com/embed/tgbNymZ7vqY"
@@ -104,15 +106,21 @@ <h2 class="service__title service__title--accordion">
               >
                 <button class="btn btn--alt btn--alt--m">
                   Print page
-                  <i aria-hidden="true" class="btn__icon btn__icon--share fas fa-print"></i>
+                  <i
+                    aria-hidden="true"
+                    class="btn__icon btn__icon--share fas fa-print"
+                  ></i>
                 </button>
               </div>
               <div
                 class="service__social--second-row col-xsmall-6 col-small-7 col-medium-12 col-large-12 col-xlarge-7 -xsmall-no-gutters"
               >
                 <button class="btn btn--alt btn--alt--m">
                   Add to favourites
-                  <i aria-hidden="true" class="btn__icon btn__icon--share fas fa-star"></i>
+                  <i
+                    aria-hidden="true"
+                    class="btn__icon btn__icon--share fas fa-star"
+                  ></i>
                 </button>
               </div>
               <div