From 99063d6e3718f62588ed6e6e96889bc2e8343d87 Mon Sep 17 00:00:00 2001
From: Simon Grim <scary@tasty.sexy>
Date: Mon, 16 Feb 2015 02:51:57 +0500
Subject: [PATCH] added focusing on tab after click on notificaton, removed
 notification in modal of mentions, fixed something

---
 js/interface_common.js       |  2 +-
 js/interface_localization.js |  4 ++--
 js/mobile_abstract.js        | 13 ++++++++++---
 js/options.js                |  2 +-
 4 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/js/interface_common.js b/js/interface_common.js
index fb0ac0d..7c19fb7 100644
--- a/js/interface_common.js
+++ b/js/interface_common.js
@@ -202,7 +202,7 @@ function updateHashtagModal(postboard,hashtag,timeoutArgs) {
     requestHashtag(postboard,hashtag,resource,timeoutArgs);
 
     if( _hashtagPendingPostsUpdated ) {
-        if ($.Options.getShowDesktopNotifPostsModalOpt() === 'enable') {
+        if (resource != 'mention' && $.Options.getShowDesktopNotifPostsModalOpt() === 'enable') {
             $.MAL.showDesktopNotif(false, polyglot.t('You got')+' '+polyglot.t("new_posts", _hashtagPendingPostsUpdated)+' '+polyglot.t('in search result')+'.', false,'twister_notification_new_posts_modal', $.Options.getShowDesktopNotifPostsModalTimerOpt(), function() {
                     $(".postboard-news").hide();
                     displayHashtagPending($(".hashtag-modal .postboard-posts"));
diff --git a/js/interface_localization.js b/js/interface_localization.js
index b19cc43..34115c0 100644
--- a/js/interface_localization.js
+++ b/js/interface_localization.js
@@ -1782,7 +1782,7 @@ if(preferredLanguage == "ru"){
        "Network status": "Состояние сети",
        "New direct message...": "Новое личное сообщение...",
        "New Post...": "Новый пост...",
-       "You got": "А у вас там",
+       "You got": "У вас",
        "in postboard": "в ленте",
        "in search result": "в результатах поиска",
        "new_posts": "%{smart_count} новый пост |||| %{smart_count} новых постов",
@@ -1791,7 +1791,7 @@ if(preferredLanguage == "ru"){
        "nobody": "Анонимно", // used to promote a post without attaching the user
        "Not available": "Недоступно",
        "notify_desktop_error": "Твистер не может выполнить уведомление: произошла неизвестная ошибка.",
-       "notify_desktop_perm_denied": "Твистер не может выполниь уведомление: разрешение не получено.\n\nЧтобы получать уведомления, разрешите их для этого домена в настройках вашего браузера.",
+       "notify_desktop_perm_denied": "Твистер не может выполнить уведомление: разрешение не получено.\n\nЧтобы получать уведомления, разрешите их для этого домена в настройках вашего браузера.",
        "notify_desktop_test": "Одна лягушка сказала:\n'если не буду квакать — лопну'.\nВы нужны нам, берегите себя.",
        "notify_desktop_title": "Watch out, it's twister over here!",
        "Number of blocks in block chain:": "Количество блоков в цепочке: ",
diff --git a/js/mobile_abstract.js b/js/mobile_abstract.js
index 2015327..fd7c426 100644
--- a/js/mobile_abstract.js
+++ b/js/mobile_abstract.js
@@ -378,7 +378,7 @@ var MAL = function()
     }
 
     this.showDesktopNotif = function(notifyTitle, notifyBody, notifyIcon, notifyTag, notifyTimer, actionOnClick, actionOnPermDenied) {
-        function doNotification(notifyTitle, notifyBody, notifyIcon, notifyTag, notifyTimer, actionOnClick) {
+        function doNotification() {
             if (!notifyTitle) {
                 notifyTitle = polyglot.t('notify_desktop_title');
             }
@@ -391,13 +391,20 @@ var MAL = function()
             if (!notifyTimer) {
                 notifyTimer = 3600 * 24 * 30; // one month
             }
+            var doActionOnClick = false;
+            if (typeof actionOnClick === 'function') {
+                doActionOnClick = function() {
+                    actionOnClick();
+                    window.focus();
+                }
+            }
 
             var desktopNotification = new Notify(notifyTitle, {
                 body: notifyBody,
                 icon: notifyIcon,
                 tag: notifyTag,
                 timeout: notifyTimer,
-                notifyClick: actionOnClick,
+                notifyClick: doActionOnClick,
                 notifyError: function() { alert(polyglot.t('notify_desktop_error')) }
             });
             desktopNotification.show();
@@ -406,7 +413,7 @@ var MAL = function()
         if (Notify.needsPermission) {
             Notify.requestPermission(false, actionOnPermDenied);
         } else {
-            doNotification(notifyTitle, notifyBody, notifyIcon, notifyTag, notifyTimer, actionOnClick);
+            doNotification();
         }
     }
 
diff --git a/js/options.js b/js/options.js
index cea7f18..467482f 100644
--- a/js/options.js
+++ b/js/options.js
@@ -212,7 +212,7 @@ var TwisterOptions = function()
 
     this.setTestDesktopNotif = function() {
         $('#testDesktopNotif').on('click', function() {
-            $.MAL.showDesktopNotif(false, polyglot.t('notify_desktop_test'), false,'twister_notification_test', false, function() { alert(polyglot.t('notify_desktop_perm_denied')) })
+            $.MAL.showDesktopNotif(false, polyglot.t('notify_desktop_test'), false,'twister_notification_test', false, false, function() { alert(polyglot.t('notify_desktop_perm_denied')) })
         })
     }