Fix for grey background flashing behind “Download All” button

jmgasper · jmgasper · commit d5acd6a78dde · 2023-02-14T07:04:39.000+11:00
diff --git a/src/shared/components/challenge-detail/Submissions/style.scss b/src/shared/components/challenge-detail/Submissions/style.scss
@@ -471,6 +471,7 @@
   justify-content: flex-end;
   margin-bottom: 10px;
 
+  button.download,
   .download {
     @include roboto-bold;
 
@@ -483,12 +484,16 @@
       color: #0ab88a;
     }
 
+    &:focus,
     &:disabled {
-      pointer-events: none;
       background-color: transparent;
       color: #137d60;
     }
 
+    &:disabled {
+      pointer-events: none;
+    }
+
     &.MM {
       cursor: pointer;
     }
diff --git a/src/shared/components/challenge-detail/Winners/style.scss b/src/shared/components/challenge-detail/Winners/style.scss
@@ -28,6 +28,7 @@
   margin-bottom: 20px;
 }
 
+button.download,
 .download {
   @include roboto-bold;
 
@@ -40,12 +41,16 @@
     color: #0ab88a;
   }
 
+  &:focus,
   &:disabled {
-    pointer-events: none;
     background-color: transparent;
     color: #137d60;
   }
 
+  &:disabled {
+    pointer-events: none;
+  }
+
   &.MM {
     cursor: pointer;
   }

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@`
`28`	`28`	`margin-bottom: 20px;`
`29`	`29`	`}`
`30`	`30`
	`31`	`+button.download,`
`31`	`32`	`.download {`
`32`	`33`	`@include roboto-bold;`
`33`	`34`
`@@ -40,12 +41,16 @@`
`40`	`41`	`color: #0ab88a;`
`41`	`42`	`}`
`42`	`43`
	`44`	`+ &:focus,`
`43`	`45`	`&:disabled {`
`44`		`- pointer-events: none;`
`45`	`46`	`background-color: transparent;`
`46`	`47`	`color: #137d60;`
`47`	`48`	`}`
`48`	`49`
	`50`	`+ &:disabled {`
	`51`	`+ pointer-events: none;`
	`52`	`+ }`
	`53`	`+`
`49`	`54`	`&.MM {`
`50`	`55`	`cursor: pointer;`
`51`	`56`	`}`