From a3644638aaf0418598196a870204e0b632a4c8ad Mon Sep 17 00:00:00 2001 From: Thomas Bruederli <thomas@roundcube.net> Date: Fri, 17 Apr 2015 06:28:40 -0400 Subject: [PATCH] Allow preference sections to define CSS class names --- plugins/newmail_notifier/newmail_notifier.js | 33 ++++++++++++++++++++++++--------- 1 files changed, 24 insertions(+), 9 deletions(-) diff --git a/plugins/newmail_notifier/newmail_notifier.js b/plugins/newmail_notifier/newmail_notifier.js index 69f1867..79911f2 100644 --- a/plugins/newmail_notifier/newmail_notifier.js +++ b/plugins/newmail_notifier/newmail_notifier.js @@ -1,8 +1,20 @@ /** * New Mail Notifier plugin script * - * @version @package_version@ * @author Aleksander Machniak <alec@alec.pl> + * + * @licstart The following is the entire license notice for the + * JavaScript code in this file. + * + * Copyright (c) 2013, The Roundcube Dev Team + * + * The JavaScript code in this page is free software: you can redistribute it + * and/or modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation, either version 3 of + * the License, or (at your option) any later version. + * + * @licend The above is the entire license notice + * for the JavaScript code in this file. */ if (window.rcmail && rcmail.env.task == 'mail') { @@ -46,12 +58,13 @@ // Basic notification: window.focus and favicon change function newmail_notifier_basic() { - var w = rcmail.is_framed() ? window.parent : window; + var w = rcmail.is_framed() ? window.parent : window, + path = rcmail.assets_path('plugins/newmail_notifier'); w.focus(); // we cannot simply change a href attribute, we must to replace the link element (at least in FF) - var link = $('<link rel="shortcut icon" href="plugins/newmail_notifier/favicon.ico"/>'), + var link = $('<link rel="shortcut icon">').attr('href', path + '/favicon.ico'), oldlink = $('link[rel="shortcut icon"]', w.document); if (!rcmail.env.favicon_href) @@ -63,7 +76,7 @@ // Add IE icon overlay if we're pinned to Taskbar try { if (window.external.msIsSiteMode()) { - window.external.msSiteModeSetIconOverlay('plugins/newmail_notifier/overlay.ico', rcmail.gettext('title', 'newmail_notifier')); + window.external.msSiteModeSetIconOverlay(path + '/overlay.ico', rcmail.gettext('title', 'newmail_notifier')); } } catch(e) {} } @@ -71,7 +84,7 @@ // Sound notification function newmail_notifier_sound() { - var elem, src = 'plugins/newmail_notifier/sound', + var elem, src = rcmail.assets_path('plugins/newmail_notifier/sound'), plugin = navigator.mimeTypes ? navigator.mimeTypes['audio/mp3'] : {}; // Internet Explorer does not support wav files, @@ -81,7 +94,7 @@ // HTML5 try { - elem = $('<audio src="' + src + '" />'); + elem = $('<audio>').attr('src', src); elem.get(0).play(); } // old method @@ -96,7 +109,9 @@ // - Require Chrome or Firefox latest version (22+) / 21.0 or older with a plugin function newmail_notifier_desktop(body) { - var timeout = rcmail.env.newmail_notifier_timeout || 10; + var timeout = rcmail.env.newmail_notifier_timeout || 10, + icon = rcmail.assets_path('plugins/newmail_notifier/mail.png'); + // As of 17 June 2013, Chrome/Chromium does not implement Notification.permission correctly that // it gives 'undefined' until an object has been created: @@ -108,7 +123,7 @@ lang: "", body: body, tag: "newmail_notifier", - icon: "plugins/newmail_notifier/mail.png" + icon: icon }); popup.onclick = function() { this.close(); @@ -123,7 +138,7 @@ if (dn && !dn.checkPermission()) { if (rcmail.newmail_popup) rcmail.newmail_popup.cancel(); - var popup = window.webkitNotifications.createNotification('plugins/newmail_notifier/mail.png', + var popup = window.webkitNotifications.createNotification(icon, rcmail.gettext('title', 'newmail_notifier'), body); popup.onclick = function() { this.cancel(); -- Gitblit v1.9.1