[bugfix/improvement] add a util class for formatting purposes

...and output formatted numbers for the client window

app/Application.js

@@ -1,5 +1,8 @@
 Ext.define('Rambox.Application', {
 	 extend: 'Ext.app.Application'
+	 ,requires: [
+ 		'Rambox.util.Format'
+ 	]
 
 	,name: 'Rambox'
 
@@ -101,7 +104,7 @@ Ext.define('Rambox.Application', {
 	,updateTotalNotifications: function( newValue, oldValue ) {
 		newValue = parseInt(newValue);
 		if ( newValue > 0 )	{
-			document.title = 'Rambox (' + newValue + ')';
+			document.title = 'Rambox (' + Rambox.util.Format.formatNumber(newValue) + ')';
 		} else {
 			document.title = 'Rambox';
 		}

app/util/Format.js

@@ -0,0 +1,15 @@
+/**
+ * Created by vsxed on 7/11/2016.
+ */
+Ext.define('Rambox.util.Format', {
+    singleton: true,
+
+    formatNumber: function(n) {
+      return n.toString().replace(/(\d)(?=(\d{3})+(?!\d))/g, "$1,");
+    },
+
+    stripNumber: function(n) {
+      return (typeof n == "number") ? n : parseInt(n.match(/\d+/g).join(""));
+    }
+
+});

app/ux/WebView.js

@@ -4,6 +4,9 @@
 Ext.define('Rambox.ux.WebView',{
 	 extend: 'Ext.panel.Panel'
 	,xtype: 'webview'
+	,requires: [
+		'Rambox.util.Format'
+	]
 
 	// private
 	,notifications: 0
@@ -67,8 +70,12 @@ Ext.define('Rambox.ux.WebView',{
 
 	,onBadgeTextChange: function( tab, badgeText, oldBadgeText ) {
 		if ( oldBadgeText === null ) oldBadgeText = 0;
+
+		oldBadgeText = Rambox.util.Format.stripNumber(oldBadgeText);
+		badgeText = Rambox.util.Format.stripNumber(badgeText);
+
 		var actualNotifications = Rambox.app.getTotalNotifications();
-		Rambox.app.setTotalNotifications(actualNotifications - parseInt(oldBadgeText) + parseInt(badgeText));
+		Rambox.app.setTotalNotifications(actualNotifications - oldBadgeText + badgeText);
 	}
 
 	,onAfterRender: function() {
@@ -125,13 +132,15 @@ Ext.define('Rambox.ux.WebView',{
 			count = count.match(/\d+/g).join(""); // Some services have special characters. Example: (•)
 			count = count ? parseInt(count) : 0;
 
+			var formattedCount = Rambox.util.Format.formatNumber(count);
+
 			switch ( me.type ) {
 				case 'messenger':
 					if ( count !== me.notifications && count > 0 ) {
 						me.notifications = count;
 					}
 					if ( count || e.title === 'Messenger' ) {
-						me.tab.setBadgeText(count);
+						me.tab.setBadgeText(formattedCount);
 					}
 					if ( e.title === 'Messenger' ) me.notifications = 0;
 					break;
@@ -140,12 +149,12 @@ Ext.define('Rambox.ux.WebView',{
 						me.notifications = count;
 					}
 					if ( count || e.title === 'Google Hangouts' ) {
-						me.tab.setBadgeText(count);
+						me.tab.setBadgeText(formattedCount);
 					}
 					if ( e.title === 'Google Hangouts' ) me.notifications = 0;
 					break;
 				default:
-					me.tab.setBadgeText(count);
+					me.tab.setBadgeText(formattedCount);
 					me.notifications = count;
 					break;
 			}