Merge pull request #6071 from luixxiul/fix-download-bar-close-button

Replaced fa-times for close button on download bar with SVG button
brave · Dec 8, 2016 · 24e5605 · 24e5605
2 parents a6c6384 + 8caa528
commit 24e5605
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 3 deletions.
diff --git a/img/toolbar/close_download_btn.svg b/img/toolbar/close_download_btn.svg
diff --git a/img/toolbar/close_download_btn_hover.svg b/img/toolbar/close_download_btn_hover.svg
diff --git a/js/components/downloadsBar.js b/js/components/downloadsBar.js
@@ -134,8 +134,7 @@ class DownloadsBar extends ImmutableComponent {
         }
       </div>
       <div className='downloadBarButtons'>
-        <Button iconClass='fa-times'
-          className='downloadButton smallButton hideButton'
+        <Button className='downloadButton'
           onClick={downloadActions.hideDownloadsToolbar} />
       </div>
     </div>

diff --git a/less/downloadBar.less b/less/downloadBar.less
@@ -132,7 +132,17 @@
     margin: auto 0;
 
     .downloadButton {
-      margin: 0 2px;
+      background: url('../img/toolbar/close_download_btn.svg') center no-repeat;
+      background-size: 14px 14px;
+      height: 18px;
+      width: 18px;
+
+      &:hover {
+        background: url('../img/toolbar/close_download_btn_hover.svg') center no-repeat;
+        background-size: 14px 14px;
+        height: 18px;
+        width: 18px;
+      }
 
       &.hideButton:not([disabled]):hover {
         background: @chromeControlsWarningBackground;