Windows: Reintroduce flash frame when we trigger a notification

signalapp · Sep 6, 2019 · 5bd9964 · 5bd9964
1 parent e011589
commit 5bd9964
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 0 deletions.
diff --git a/js/notifications.js b/js/notifications.js
@@ -1,6 +1,7 @@
 /* global Signal:false */
 /* global Backbone: false */
 
+/* global drawAttention: false */
 /* global i18n: false */
 /* global isFocused: false */
 /* global Signal: false */
@@ -135,6 +136,8 @@
         message = i18n('newMessage');
       }
 
+      drawAttention();
+
       this.lastNotification = new Notification(title, {
         body: window.platform === 'linux' ? filter(message) : message,
         icon: iconUrl,

diff --git a/main.js b/main.js
@@ -890,6 +890,12 @@ ipc.on('add-setup-menu-items', () => {
   });
 });
 
+ipc.on('draw-attention', () => {
+  if (process.platform === 'win32' && mainWindow) {
+    mainWindow.flashFrame(true);
+  }
+});
+
 ipc.on('restart', () => {
   app.relaunch();
   app.quit();

diff --git a/preload.js b/preload.js
@@ -90,6 +90,10 @@ window.open = () => null;
 // eslint-disable-next-line no-eval, no-multi-assign
 window.eval = global.eval = () => null;
 
+window.drawAttention = () => {
+  window.log.info('draw attention');
+  ipc.send('draw-attention');
+};
 window.showWindow = () => {
   window.log.info('show window');
   ipc.send('show-window');