Merge pull request #771 from rust-lang/uniform-spinner

Use octicon spinner in status bar item

Author: Xanewok

CHANGELOG.md

@@ -1,5 +1,7 @@
 ### Unreleased
 
+* Use smooth, universally supported spinner in the status bar ⚙️
+
 ### 0.7.3 - 2020-04-21
 
 * Remove redundant `rust-client.nestedMultiRootConfigInOutermost` setting (originally used to work around non-multi-project limitations)

src/spinner.ts

@@ -1,24 +1,9 @@
 import { window } from 'vscode';
 
 export function startSpinner(prefix: string, postfix: string) {
-  if (spinnerTimer != null) {
-    clearInterval(spinnerTimer);
-  }
-  let state = 0;
-  spinnerTimer = setInterval(() => {
-    window.setStatusBarMessage(`${prefix} ${spinner[state]} ${postfix}`);
-    state = (state + 1) % spinner.length;
-  }, 100);
+  window.setStatusBarMessage(`${prefix} $(settings-gear~spin) ${postfix}`);
 }
 
 export function stopSpinner(message: string) {
-  if (spinnerTimer) {
-    clearInterval(spinnerTimer);
-  }
-  spinnerTimer = null;
-
   window.setStatusBarMessage(message);
 }
-
-let spinnerTimer: NodeJS.Timer | null = null;
-const spinner = ['🌕', '🌖', '🌗', '🌘', '🌑', '🌒', '🌓', '🌔'];