Merge pull request #200 from jnugh/linuxUrgentHint

Urgent hint on linux

CHANGELOG.md

@@ -4,6 +4,11 @@
 
 ### Improvements
 
+#### Linux
+ - Added an option to make the taskbar icon flash on new messages
+
+### Improvements
+
 ### Bug Fixes
 
 

src/browser/settings.jsx

@@ -202,7 +202,7 @@ var SettingsPage = React.createClass({
       ) : null;
 
     var notifications_row = null;
-    if (process.platform === 'win32') {
+    if (process.platform === 'win32' || process.platform === 'linux') {
       var notificationSettings = [
         {
           label: 'Never',

src/main.js

@@ -261,10 +261,13 @@ app.on('ready', function() {
       mainWindow.focus();
     });
     ipcMain.on('notified', function(event, arg) {
-      if (process.platform === 'win32') {
+      if (process.platform === 'win32' || process.platform === 'linux') {
         if (config.notifications.flashWindow === 2) {
           mainWindow.flashFrame(true);
         }
+      }
+
+      if (process.platform === 'win32') {
         // On Windows 8.1 and Windows 8, a shortcut with a Application User Model ID must be installed to the Start screen.
         // In current version, use tray balloon for notification
         if (osVersion.isLowerThanOrEqualWindows8_1()) {

test/specs/browser/settings_test.js

@@ -172,8 +172,8 @@ describe('browser/settings.html', function() {
     });
 
     describe('Notifications', function() {
-      it('should appear on win32', function() {
+      it('should appear on win32 and linux', function() {
-        const expected = (process.platform === 'win32');
+        const expected = (process.platform === 'win32' || process.platform === 'linux');
         env.addClientCommands(this.app.client);
         return this.app.client
           .loadSettingsPage()