From 1e56e46724ce7e9997a7d715c4d43b6b146b452c Mon Sep 17 00:00:00 2001 From: jdlrobson Date: Mon, 29 Jan 2018 10:22:09 -0800 Subject: [PATCH] Fix download icon spinner Regression, introduced in I6e474562ba5f2c704364c9d7b49f982f7bab8a18 Bug: T182162 Change-Id: I56464d42e437a94b2008338b00f49b4e0b086561 --- resources/skins.minerva.scripts/DownloadIcon.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/resources/skins.minerva.scripts/DownloadIcon.js b/resources/skins.minerva.scripts/DownloadIcon.js index ea6a04389..cd9f350cd 100644 --- a/resources/skins.minerva.scripts/DownloadIcon.js +++ b/resources/skins.minerva.scripts/DownloadIcon.js @@ -80,6 +80,10 @@ * Replace download icon with a spinner */ showSpinner: function () { + // FIXME: There is no spinner icon in Minerva, only in MobileFrontend + // Hopefully when T177432 is resolved this and corresponding change in hideSpinner + // should be unnecessary. + this.options.glyphPrefix = 'mf'; this.options.name = 'spinner'; this.render(); }, @@ -87,6 +91,7 @@ * Restore download icon from spinner state */ hideSpinner: function () { + this.options.glyphPrefix = 'minerva'; this.options.name = GLYPH; this.render(); },