🔖 v0.5.9

updated contributing guide

.travis.yml

@@ -5,6 +5,14 @@ os:
 - osx
 - linux
 
+addons:
+  apt:
+    packages:
+    - build-essential
+    - libxext-dev
+    - libxtst-dev
+    - libxkbfile-dev
+
 language: node_js
 node_js:
 - '6'
@@ -22,3 +30,9 @@ cache:
 
 script:
 - npm run travis
+notifications:
+  webhooks:
+    urls:
+      - https://zulip.org/zulipbot/travis
+    on_success: always
+    on_failure: always

CONTRIBUTING.md

@@ -1,16 +1,24 @@
-#Contributing Guidelines
+# Contributing Guidelines
 
 Thanks for taking the time to contribute!
 
-The following is a set of guidelines for contributing to zulip-electron. These are just guidelines, not rules, use your best judgment and feel free to propose changes to this document in a pull request.
+The following is a set of guidelines for contributing to zulip-electron. These are just guidelines, not rules, use your best judgement and feel free to propose changes to this document in a pull request.
 
 ## Getting Started
 
 Zulip-Desktop app is built on top of [Electron](http://electron.atom.io/). If you are new to Electron please head over to [this](http://jlord.us/essential-electron/) great article.
 
+## Community
+
+* The whole zulip documentation such as development environment, setting up with zulip project, testing, code of conduct can be read [here](https://zulip.readthedocs.io).
+
+* If you have any questions regarding zulip-electron, open an [issue](https://github.com/zulip/zulip-electron/issues/new/) or ask it [here](https://chat.zulip.org/#narrow/stream/electron).
+
 ## Issue
 Ensure the bug was not already reported by searching on GitHub under [Issues](https://github.com/zulip/zulip-electron/issues). If you're unable to find an open issue addressing the problem, [open a new one](https://github.com/zulip/zulip-electron/issues/new). Please pay attention to following points while opening an issue.
 
+The [Zulipbot](https://github.com/zulip/zulipbot) helps to claim the issue by commenting the following in the comment section: "**@zulipbot** claim". **@zulipbot** will assign you to the issue and label the issue as **in progress**. For more details, check out [**@zulipbot**](https://github.com/zulip/zulipbot).
+
 ### Does it happen on web browsers? (especially Chrome)
 Zulip-Desktop is based on Electron, which integrates the Chrome engine within a standalone application.
 If the problem you encounter can be reproduced on web browsers, it may be an issue with Zulip web app.

README.md

@@ -5,7 +5,7 @@
 
 This is an experimental replacement for the [Zulip Desktop
 app](https://github.com/zulip/zulip-desktop) implemented in
-[Electron](http://electron.atom.io/).
+[Electron](http://electron.atom.io/).eeee
 
 The goal is to achieve feature-compatibility with the old desktop app
 and then start adding cool features like easy support for
@@ -13,11 +13,13 @@ multi-account, auto-updates etc.
 
 ## Prerequisites
 * node >= v6.3.1
-* npm >= 3.10.3
+> Use [nvm](https://github.com/creationix/nvm) to install the current stable version of node
+
+
 * python (v2.7.x recommended)
 * If you're on Debian or Ubuntu, you'll need to install following packages:
 ```sh
-$ sudo apt-get install nodejs-legacy build-essential libxext-dev libxtst-dev libxkbfile-dev
+$ sudo apt-get install build-essential libxext-dev libxtst-dev libxkbfile-dev
 ```
 
 ## Installation

app/main/autoupdater.js

@@ -1,30 +1,21 @@
 'use strict';
-// const os = require('os');
 const {app, dialog} = require('electron');
 const {autoUpdater} = require('electron-updater');
 
-// We don't need to call all of these since it's automatically handled by electron-updater
-// const version = app.getVersion();
-// const platform = os.platform() + '_' + os.arch();  // usually returns darwin_64
-// const updaterFeedURL = 'http://zulipdesktop.herokuapp.com/update/' + platform + '/' + version;
-
 function appUpdater() {
-	// autoUpdater.setFeedURL(updaterFeedURL);
-
 	// Log whats happening
 	const log = require('electron-log');
 	log.transports.file.level = 'info';
 	autoUpdater.logger = log;
-	// TODO send autoUpdater events to renderer so that we could
-	// it could console log in developer tools
 	/*
-	autoUpdater.on('error', err => console.log(err));
+	autoUpdater.on('error', err => log.info(err));
-	autoUpdater.on('checking-for-update', () => console.log('checking-for-update'));
+	autoUpdater.on('checking-for-update', () => log.info('checking-for-update'));
-	autoUpdater.on('update-available', () => console.log('update-available'));
+	autoUpdater.on('update-available', () => log.info('update-available'));
-	autoUpdater.on('update-not-available', () => console.log('update-not-available'));
+	autoUpdater.on('update-not-available', () => log.info('update-not-available'));
 	*/
 
 	// Ask the user if update is available
+	// eslint-disable-next-line no-unused-vars
 	autoUpdater.on('update-downloaded', (event, info) => {
 		// let message = app.getName() + ' ' + info.releaseName + ' is now available. It will be installed the next time you restart the application.';
 		// if (info.releaseNotes) {

app/main/domain.js

@@ -7,30 +7,42 @@ const db = new JsonDB(app.getPath('userData') + '/domain.json', true, true);
 const data = db.getData('/');
 
 console.log(data.domain);
-
-// if (data.domain && window.location.href.indexOf(data.domain) === -1) {
-// 	window.location.href = data.domain
-// }
-// require('electron-connect').client.create();
-
 window.addDomain = function () {
-	document.getElementById('main').innerHTML = 'checking...';
-
 	let newDomain = document.getElementById('url').value;
 	newDomain = newDomain.replace(/^https?:\/\//, '');
-
+	newDomain = newDomain.replace(/^http?:\/\//, '');
-	const domain = 'https://' + newDomain;
+	if (newDomain === '') {
-	const checkDomain = domain + '/static/audio/zulip.ogg';
+		document.getElementById('server-status').innerHTML = 'Please input a value';
-
+	} else {
-	request(checkDomain, (error, response) => {
+		document.getElementById('main').innerHTML = 'Checking...';
-		if (!error && response.statusCode !== 404) {
+		if (newDomain.indexOf('localhost:') >= 0) {
-			document.getElementById('main').innerHTML = 'Connect';
+			const domain = 'http://' + newDomain;
-			db.push('/domain', domain);
+			const checkDomain = domain + '/static/audio/zulip.ogg';
-			ipcRenderer.send('new-domain', domain);
+			request(checkDomain, (error, response) => {
+				if (!error && response.statusCode !== 404) {
+					document.getElementById('main').innerHTML = 'Connect';
+					db.push('/domain', domain);
+					ipcRenderer.send('new-domain', domain);
+				} else {
+					document.getElementById('main').innerHTML = 'Connect';
+					document.getElementById('server-status').innerHTML = 'Not a valid Zulip Local Server.';
+				}
+			});
+			//	});
 		} else {
-			document.getElementById('main').innerHTML = 'Connect';
+			const domain = 'https://' + newDomain;
-			document.getElementById('server-status').innerHTML = 'Not a valid Zulip Server.';
+			const checkDomain = domain + '/static/audio/zulip.ogg';
-		}
-	});
-};
 
+			request(checkDomain, (error, response) => {
+				if (!error && response.statusCode !== 404) {
+					document.getElementById('main').innerHTML = 'Connect';
+					db.push('/domain', domain);
+					ipcRenderer.send('new-domain', domain);
+				} else {
+					document.getElementById('main').innerHTML = 'Connect';
+					document.getElementById('server-status').innerHTML = 'Not a valid Zulip Server.';
+				}
+			});
+		}
+	}
+};

app/main/index.js

@@ -1,11 +1,15 @@
 'use strict';
 const path = require('path');
 const fs = require('fs');
+const os = require('os');
 const electron = require('electron');
 const {app} = require('electron');
 const ipc = require('electron').ipcMain;
+const {dialog} = require('electron');
+const https = require('https');
+const http = require('http');
 const electronLocalshortcut = require('electron-localshortcut');
-const Configstore = require('configstore');
+const Configstore = require('electron-config');
 const JsonDB = require('node-json-db');
 const isDev = require('electron-is-dev');
 const tray = require('./tray');
@@ -16,16 +20,29 @@ const {appUpdater} = require('./autoupdater');
 const db = new JsonDB(app.getPath('userData') + '/domain.json', true, true);
 const data = db.getData('/');
 
-/* Handling squirrel.windows events on windows
-Update - Not needed for NSIS
-if (require('electron-squirrel-startup')) {
-	app.quit();
-} */
-
 // adds debug features like hotkeys for triggering dev tools and reload
 require('electron-debug')();
 
-const conf = new Configstore('Zulip-Desktop');
+const conf = new Configstore();
+
+function userOS() {
+	if (os.platform() === 'darwin') {
+		return 'Mac';
+	}
+	if (os.platform() === 'linux') {
+		return 'Linux';
+	}
+	if (os.platform() === 'win32' || os.platform() === 'win64') {
+		if (parseFloat(os.release()) < 6.2) {
+			return 'Windows 7';
+		} else {
+			return 'Windows 10';
+		}
+	}
+}
+
+// setting userAgent so that server-side code can identify the desktop app
+const isUserAgent = 'ZulipElectron/' + app.getVersion() + ' ' + userOS();
 
 // prevent window being garbage collected
 let mainWindow;
@@ -41,6 +58,58 @@ const targetURL = function () {
 	return data.domain;
 };
 
+function serverError(targetURL) {
+	if (targetURL.indexOf('localhost:') < 0 && data.domain) {
+		const req = https.request(targetURL + '/static/audio/zulip.ogg', res => {
+			console.log('Server StatusCode:', res.statusCode);
+			console.log('You are connected to:', res.req._headers.host);
+			if (res.statusCode >= 500 && res.statusCode <= 599) {
+				return dialog.showErrorBox('SERVER IS DOWN!', 'We are getting a ' + res.statusCode + ' error status from the server ' + res.req._headers.host + '. Please try again after some time or you may switch server.');
+			}
+		});
+		req.on('error', e => {
+			console.error(e);
+		});
+		req.end();
+	} else if (data.domain) {
+		const req = http.request(targetURL + '/static/audio/zulip.ogg', res => {
+			console.log('Server StatusCode:', res.statusCode);
+			console.log('You are connected to:', res.req._headers.host);
+		});
+		req.on('error', e => {
+			console.error(e);
+		});
+		req.end();
+	}
+}
+
+function checkConnectivity() {
+	return dialog.showMessageBox({
+		title: 'Internet connection problem',
+		message: 'No internet available! Try again?',
+		type: 'warning',
+		buttons: ['Try again', 'Close'],
+		defaultId: 0
+	}, index => {
+		if (index === 0) {
+			mainWindow.webContents.reload();
+		}
+		if (index === 1) {
+			app.quit();
+		}
+	});
+}
+
+function checkConnection() {
+	// eslint-disable-next-line no-unused-vars
+	mainWindow.webContents.on('did-fail-load', (event, errorCode, errorDescription, validatedURL) => {
+		if (errorDescription === 'ERR_INTERNET_DISCONNECTED' || errorDescription === 'ERR_PROXY_CONNECTION_FAILED') {
+			console.log('Error Description:' + errorDescription);
+			checkConnectivity();
+		}
+	});
+}
+
 const isAlreadyRunning = app.makeSingleInstance(() => {
 	if (mainWindow) {
 		if (mainWindow.isMinimized()) {
@@ -73,8 +142,8 @@ const iconPath = () => {
 };
 
 function onClosed() {
-	// dereference the window
+	// Dereference the window
-	// for multiple windows store them in an array
+	// For multiple windows, store them in an array
 	mainWindow = null;
 }
 
@@ -105,10 +174,20 @@ function createMainWindow() {
 			plugins: true,
 			allowDisplayingInsecureContent: true,
 			nodeIntegration: false
-		}
+		},
+		show: false
+	});
+
+	win.once('ready-to-show', () => {
+		win.show();
+	});
+
+	serverError(targetURL());
+
+	win.loadURL(targetURL(), {
+		userAgent: isUserAgent + ' ' + win.webContents.getUserAgent()
 	});
 
-	win.loadURL(targetURL());
 	win.on('closed', onClosed);
 	win.setTitle('Zulip');
 
@@ -176,7 +255,7 @@ app.on('activate', () => {
 app.on('ready', () => {
 	electron.Menu.setApplicationMenu(appMenu);
 	mainWindow = createMainWindow();
-	tray.create(mainWindow);
+	tray.create();
 
 	const page = mainWindow.webContents;
 
@@ -214,10 +293,11 @@ app.on('ready', () => {
 	page.once('did-frame-finish-load', () => {
 		const checkOS = isWindowsOrmacOS();
 		if (checkOS && !isDev) {
-			// Initate auto-updates on macOs and windows
+			// Initate auto-updates on MacOS and Windows
 			appUpdater();
 		}
 	});
+	checkConnection();
 });
 
 app.on('will-quit', () => {
@@ -235,6 +315,7 @@ ipc.on('new-domain', (e, domain) => {
 		mainWindow.show();
 	} else {
 		mainWindow.loadURL(domain);
+		serverError(domain);
 	}
 	targetLink = domain;
 });

app/main/macos-swipe-navigation.js

@@ -1,69 +0,0 @@
-// Adopted from
-// https://github.com/wozaki/twitter-js-apps/blob/9bc00eafd575fd180dc7a450e1b1daf425e67b80/redux/src/main/renderer/registries/electron/swipeNavigatorImpl.js
-
-'use strict'
-const {remote} = require('electron');
-
-const THRESHOLD_DELTA_X = 70;
-const THRESHOLD_LIMIT_DELTA_Y = 50;
-const THRESHOLD_TIME = 50;
-
-// TODO avoid module global state
-let tracking = false;
-let deltaX = 0;
-let deltaY = 0;
-let startTime = 0;
-let time = 0;
-
-module.exports.register = function register() {
-  remote.getCurrentWindow()
-    .on('scroll-touch-begin', onScrollBegin)
-    .on('scroll-touch-end', onScrollEnd)
-    .on('swipe', onSwipe)
-
-  window.addEventListener('wheel', onMouseWheel, {passive: true});
-  window.addEventListener('beforeunload', remove);
-}
-
-const remove = module.exports.remove = function remove() {
-  remote.getCurrentWindow()
-    .removeListener('scroll-touch-begin', onScrollBegin)
-    .removeListener('scroll-touch-end', onScrollEnd)
-    .removeListener('swipe', onSwipe)
-
-  window.removeEventListener('mousewheel', onMouseWheel);
-  window.removeEventListener('beforeunload', remove);
-}
-
-function onSwipe(e, direction) {
-  if (direction === 'left')
-    remote.getCurrentWebContents().goBack()
-  else if (direction === 'right')
-    remote.getCurrentWebContents().goForward()
-}
-
-function onMouseWheel(e) {
-  if (tracking) {
-    deltaX = deltaX + e.deltaX
-    deltaY = deltaY + e.deltaY
-    time   = (new Date()).getTime() - startTime
-  }
-}
-
-function onScrollBegin() {
-  tracking  = true
-  startTime = (new Date()).getTime()
-}
-
-function onScrollEnd() {
-  if (time > THRESHOLD_TIME && tracking && Math.abs(deltaY) < THRESHOLD_LIMIT_DELTA_Y)
-    if (deltaX > THRESHOLD_DELTA_X)
-      remote.getCurrentWebContents().goForward()
-    else if (deltaX < -THRESHOLD_DELTA_X)
-      remote.getCurrentWebContents().goBack()
-
-  tracking = false
-  deltaX = 0
-  deltaY = 0
-  startTime = 0
-}

app/main/menu.js

@@ -8,6 +8,7 @@ const app = electron.app;
 const BrowserWindow = electron.BrowserWindow;
 const shell = electron.shell;
 const appName = app.getName();
+const tray = require('./tray');
 
 const {addDomain, about} = require('./windowmanager');
 
@@ -74,6 +75,14 @@ const viewSubmenu = [
 	{
 		type: 'separator'
 	},
+	{
+		label: 'Toggle Tray Icon',
+		click(item, focusedWindow) {
+			if (focusedWindow) {
+				tray.toggle();
+			}
+		}
+	},
 	{
 		label: 'Toggle Developer Tools',
 		accelerator: process.platform === 'darwin' ? 'Alt+Command+I' : 'Ctrl+Shift+I',

app/main/preload.js

@@ -3,8 +3,12 @@ const ipcRenderer = require('electron').ipcRenderer;
 const {webFrame} = require('electron');
 const {spellChecker} = require('./spellchecker');
 
-// enable swipe back/forward navigation on macOS
+const _setImmediate = setImmediate;
-require('./macos-swipe-navigation.js').register();
+const _clearImmediate = clearImmediate;
+process.once('loaded', () => {
+	global.setImmediate = _setImmediate;
+	global.clearImmediate = _clearImmediate;
+});
 
 // eslint-disable-next-line import/no-unassigned-import
 require('./domain');
@@ -45,10 +49,14 @@ ipcRenderer.on('log-out', () => {
 
 ipcRenderer.on('shortcut', () => {
 	// create the menu for the below
-	document.querySelector('.dropdown-toggle').click();
+	const node = document.querySelector('a[data-overlay-trigger=keyboard-shortcuts]');
-
+	// additional check
-	const nodes = document.querySelectorAll('.dropdown-menu li:nth-child(4) a');
+	if (node.text.trim().toLowerCase() === 'keyboard shortcuts') {
-	nodes[nodes.length - 1].click();
+		node.click();
+	} else {
+		// atleast click the dropdown
+		document.querySelector('.dropdown-toggle').click();
+	}
 });
 
 // To prevent failing this script on linux we need to load it after the document loaded

app/main/tray.js

@@ -15,7 +15,7 @@ const iconPath = () => {
 	return APP_ICON + (process.platform === 'win32' ? 'win.ico' : 'osx.png');
 };
 
-exports.create = () => {
+const createHandler = () => {
 	const contextMenu = electron.Menu.buildFromTemplate([
 		{
 			label: 'About',
@@ -59,3 +59,23 @@ exports.create = () => {
 	tray.setContextMenu(contextMenu);
 };
 
+const destroyHandler = () => {
+	tray.destroy();
+	if (tray.isDestroyed()) {
+		tray = null;
+	} else {
+		throw new Error('Tray icon not properly destroyed.');
+	}
+};
+
+const toggleHandler = () => {
+	if (tray) {
+		destroyHandler();
+	} else {
+		createHandler();
+	}
+};
+
+exports.create = createHandler;
+exports.destroy = destroyHandler;
+exports.toggle = toggleHandler;

app/main/windowmanager.js

@@ -2,6 +2,11 @@
 const path = require('path');
 const electron = require('electron');
 
+const APP_ICON = path.join(__dirname, '../resources', 'Icon');
+
+const iconPath = () => {
+	return APP_ICON + (process.platform === 'win32' ? '.ico' : '.png');
+};
 let domainWindow;
 let aboutWindow;
 
@@ -14,10 +19,14 @@ function onClosed() {
 // Change Zulip server Window
 function createdomainWindow() {
 	const domainwin = new electron.BrowserWindow({
+		title: 'Switch Server',
 		frame: false,
 		height: 300,
 		resizable: false,
-		width: 400
+		width: 400,
+		show: false,
+		icon: iconPath()
+
 	});
 	const domainURL = 'file://' + path.join(__dirname, '../renderer', 'pref.html');
 	domainwin.loadURL(domainURL);
@@ -25,13 +34,20 @@ function createdomainWindow() {
 
 	return domainwin;
 }
-
 // Call this window onClick addDomain in tray
 function addDomain() {
 	domainWindow = createdomainWindow();
-	domainWindow.show();
+	domainWindow.once('ready-to-show', () => {
+		domainWindow.show();
+	});
+	setTimeout(() => {
+		if (domainWindow !== null) {
+			if (!domainWindow.isDestroyed()) {
+				domainWindow.destroy();
+			}
+		}
+	}, 15000);
 }
-
 // About window
 function createAboutWindow() {
 	const aboutwin = new electron.BrowserWindow({
@@ -61,9 +77,9 @@ function createAboutWindow() {
 // Call this onClick About in tray
 function about() {
 	aboutWindow = createAboutWindow();
-	aboutWindow.show();
+	aboutWindow.once('ready-to-show', () => {
+		aboutWindow.show();
+	});
 }
 
-exports = module.exports = {
+exports = module.exports = {addDomain, about};
-	addDomain, about
-};

app/package.json

@@ -1,11 +1,11 @@
 {
   "name": "zulip",
   "productName": "Zulip",
-  "version": "0.5.7-beta",
+  "version": "0.5.9",
   "description": "Zulip Desktop App",
   "license": "Apache-2.0",
-  "email":"<svnitakash@gmail.com>",
+  "email": "<svnitakash@gmail.com>",
-  "copyright": "©2016 Kandra Labs, Inc.",
+  "copyright": "©2017 Kandra Labs, Inc.",
   "author": {
     "name": "Akash Nimare",
     "email": "svnitakash@gmail.com"
@@ -27,16 +27,16 @@
     "InstantMessaging"
   ],
   "dependencies": {
-    "electron-is-dev": "0.1.2",
+    "electron-config":"0.2.1",
-    "electron-updater": "1.4.1",
-    "electron-log": "1.3.0",
-    "configstore": "2.1.0",
-    "dialogs": "1.1.14",
     "electron-debug": "1.1.0",
-    "electron-localshortcut": "0.6.1",
+    "electron-is-dev": "0.1.2",
+    "electron-localshortcut": "1.0.0",
+    "electron-log": "1.3.0",
+    "electron-spellchecker": "1.0.5",
+    "electron-updater": "1.11.2",
+    "https": "^1.0.0",
     "node-json-db": "0.7.3",
     "request": "2.79.0",
-    "electron-spellchecker": "1.0.0",
     "wurl": "2.1.0"
   }
 }

app/renderer/css/pref.css

@@ -71,13 +71,4 @@ button:focus {
     top: -61%;
     left: 25%;
     text-align: center;
-}
-
-#pic {
-width: 20px;
-left: 36%;
-margin-left: 4px;
-top: 63%;
-display: none;
-position: absolute;
 }

app/renderer/js/pref.js

@@ -14,35 +14,54 @@ document.addEventListener('keydown', event => {
 	}
 });
 // eslint-disable-next-line no-unused-vars
-function addDomain() {
+window.prefDomain = function () {
 	const request = require('request');
 	// eslint-disable-next-line import/no-extraneous-dependencies
 	const ipcRenderer = require('electron').ipcRenderer;
 	const JsonDB = require('node-json-db');
 	// eslint-disable-next-line import/no-extraneous-dependencies
-	const {app} = require('electron').remote;
+	const {
+		app
+	} = require('electron').remote;
 
 	const db = new JsonDB(app.getPath('userData') + '/domain.json', true, true);
-	document.getElementById('main').innerHTML = 'checking...';
-	document.getElementById('pic').style.display = 'block';
 
 	let newDomain = document.getElementById('url').value;
 	newDomain = newDomain.replace(/^https?:\/\//, '');
+	newDomain = newDomain.replace(/^http?:\/\//, '');
 
-	const domain = 'https://' + newDomain;
+	if (newDomain === '') {
-	const checkDomain = domain + '/static/audio/zulip.ogg';
+		document.getElementById('urladded').innerHTML = 'Please input a value';
-
+	} else {
-	request(checkDomain, (error, response) => {
+		document.getElementById('main').innerHTML = 'Checking...';
-		if (!error && response.statusCode !== 404) {
+		if (newDomain.indexOf('localhost:') >= 0) {
-			document.getElementById('pic').style.display = 'none';
+			const domain = 'http://' + newDomain;
-			document.getElementById('main').innerHTML = 'Switch';
+			const checkDomain = domain + '/static/audio/zulip.ogg';
-			document.getElementById('urladded').innerHTML = 'Switched to ' + newDomain;
+			request(checkDomain, (error, response) => {
-			db.push('/domain', domain);
+				if (!error && response.statusCode !== 404) {
-			ipcRenderer.send('new-domain', domain);
+					document.getElementById('main').innerHTML = 'Switch';
+					document.getElementById('urladded').innerHTML = 'Switched to ' + newDomain;
+					db.push('/domain', domain);
+					ipcRenderer.send('new-domain', domain);
+				} else {
+					document.getElementById('main').innerHTML = 'Switch';
+					document.getElementById('urladded').innerHTML = 'Not a valid Zulip Local Server.';
+				}
+			});
 		} else {
-			document.getElementById('pic').style.display = 'none';
+			const domain = 'https://' + newDomain;
-			document.getElementById('main').innerHTML = 'Switch';
+			const checkDomain = domain + '/static/audio/zulip.ogg';
-			document.getElementById('urladded').innerHTML = 'Not a valid Zulip Server.';
+			request(checkDomain, (error, response) => {
+				if (!error && response.statusCode !== 404) {
+					document.getElementById('main').innerHTML = 'Switch';
+					document.getElementById('urladded').innerHTML = 'Switched to ' + newDomain;
+					db.push('/domain', domain);
+					ipcRenderer.send('new-domain', domain);
+				} else {
+					document.getElementById('main').innerHTML = 'Switch';
+					document.getElementById('urladded').innerHTML = 'Not a valid Zulip Server.';
+				}
+			});
 		}
-	});
+	}
-}
+};

app/renderer/pref.html

@@ -3,17 +3,14 @@
 	<head>
 		<meta charset="UTF-8">
 		<link rel="stylesheet" href="css/pref.css">
-		<script type="text/javascript">
-		</script>
 	</head>
 	<body>
 		<div class="close" id="close-button">Close</div>
 		<div class="form">
-			<form onsubmit="addDomain(); return false">
+			<form onsubmit="prefDomain(); return false">
 				<input id="url" type="text" placeholder="Server URL">
-				<button type="submit" id="main" value="Submit" onclick="addDomain();">
+				<button type="submit" id="main" value="Submit">
 				Switch</button>
-				<img id="pic" src="img/loader.gif" />
 			</form>
 			<p id="urladded"><p>
 			</div>

package.json

@@ -1,11 +1,12 @@
 {
   "name": "zulip",
   "productName": "Zulip",
-  "version": "0.5.7-beta",
+  "version": "0.5.9",
+  "main": "./app/main",
   "description": "Zulip Desktop App",
   "license": "Apache-2.0",
-  "email":"<svnitakash@gmail.com>",
+  "email": "<svnitakash@gmail.com>",
-  "copyright": "©2016 Kandra Labs, Inc.",
+  "copyright": "©2017 Kandra Labs, Inc.",
   "author": {
     "name": "Akash Nimare",
     "email": "svnitakash@gmail.com"
@@ -24,30 +25,33 @@
     "dev": "gulp dev",
     "pack": "build --dir",
     "dist": "build",
+    "build:win": "build --win nsis-web --ia32 --x64",
     "travis": "cd ./scripts && ./travis-build-test.sh"
   },
   "build": {
     "appId": "org.zulip.zulip-electron",
-    "asar": "true",
+    "asar": true,
     "files": [
       "**/*",
       "!node_modules/@paulcbetts/cld/deps/cld${/*}"
     ],
-    "copyright": "©2016 Kandra Labs, Inc.",
+    "copyright": "©2017 Kandra Labs, Inc.",
     "mac": {
       "category": "public.app-category.productivity"
     },
-    "linux" : {
+    "linux": {
-      "synopsis": "Zulip Desktop App",
       "category": "",
       "packageCategory": "GNOME;GTK;Network;InstantMessaging",
       "description": "Zulip Desktop Client for Linux",
-      "target" : ["deb", "AppImage"],
+      "target": [
-      "version" : "0.5.7-beta",
+        "deb",
-      "title" : "Zulip",
+        "AppImage"
-      "license": "Apache-2.0",
+      ],
       "maintainer": "Akash Nimare <svnitakash@gmail.com>"
     },
+    "deb": {
+      "synopsis": "Zulip Desktop App"
+    },
     "dmg": {
       "background": "build/appdmg.png",
       "icon": "build/icon.icns",
@@ -68,27 +72,26 @@
     },
     "win": {
       "target": "nsis",
-      "icon": "build/icon.ico",
+      "icon": "build/icon.ico"
-      "iconUrl": "https://raw.githubusercontent.com/zulip/zulip-electron/master/build/icon.ico"
+      },
-    },
     "nsis": {
       "perMachine": true,
       "oneClick": false
     }
   },
   "keywords": [
-  	"Zulip",
+    "Zulip",
-  	"Group Chat app",
+    "Group Chat app",
-  	"electron-app",
+    "electron-app",
-  	"electron",
+    "electron",
-  	"Desktop app",
+    "Desktop app",
-  	"InstantMessaging"
+    "InstantMessaging"
   ],
   "devDependencies": {
     "assert": "^1.4.1",
     "devtron": "^1.1.0",
-    "electron-builder": "13.0.0",
+    "electron-builder": "16.6.0",
-    "electron": "1.4.7",
+    "electron": "1.4.15",
     "electron-connect": "^0.4.6",
     "gulp": "^3.9.1",
     "gulp-mocha": "^3.0.1",
@@ -108,7 +111,7 @@
           "no-warning-comments": 0,
           "no-else-return": 0,
           "import/no-unresolved": 0,
-          "import/no-extraneous-dependencies":0
+          "import/no-extraneous-dependencies": 0
         }
       }
     ],

tests/index.js

@@ -7,7 +7,7 @@ describe('application launch', function () {
   beforeEach(function () {
     this.app = new Application({
       path: require('electron'),
-      args: [__dirname + '/../app/main/index.js']
+      args: [__dirname + '/../app/renderer/index.html']
     })
     return this.app.start()
   })