From 799e86de4a7db218a8bc3a3739b2fded74356f05 Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Thu, 14 Apr 2016 22:58:51 -0400
Subject: [PATCH] update form

---
 dashboard-ui/notificationsetting.html       |  9 ++++-----
 dashboard-ui/scripts/notificationsetting.js | 20 ++++++++++++--------
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/dashboard-ui/notificationsetting.html b/dashboard-ui/notificationsetting.html
index c60babe11f..1e4d61b80b 100644
--- a/dashboard-ui/notificationsetting.html
+++ b/dashboard-ui/notificationsetting.html
@@ -1,21 +1,20 @@
-<div id="notificationSettingPage" data-role="page" class="page type-interior notificationConfigurationPage withTabs" data-helpurl="https://github.com/MediaBrowser/Wiki/wiki/Notifications" data-require="scripts/notificationsetting,paper-input,jqmcheckbox">
+<div id="notificationSettingPage" data-role="page" class="page type-interior notificationConfigurationPage withTabs" data-helpurl="https://github.com/MediaBrowser/Wiki/wiki/Notifications" data-require="scripts/notificationsetting,paper-input,paper-checkbox">
 
     <div data-role="content">
         <div class="content-primary">
 
-            <h2 class="notificationType" style="margin-bottom: 0;"></h2>
+            <h1 class="notificationType" style="margin-top:0;"></h1>
 
             <form class="notificationSettingForm">
 
                 <ul data-role="listview" class="ulForm">
                     <li>
-                        <input type="checkbox" id="chkEnabled" data-mini="true" />
-                        <label for="chkEnabled">${LabelNotificationEnabled}</label>
+                        <paper-checkbox id="chkEnabled">${LabelNotificationEnabled}</paper-checkbox>
                     </li>
                 </ul>
 
                 <div class="monitorUsers" style="display: none;">
-                    <label>${LabelMonitorUsers}</label>
+                    <div class="paperCheckboxListLabel">${LabelMonitorUsers}</div>
                     <div class="monitorUsersList">
                     </div>
                     <br />
diff --git a/dashboard-ui/scripts/notificationsetting.js b/dashboard-ui/scripts/notificationsetting.js
index a0237c47f4..478ee20ee3 100644
--- a/dashboard-ui/scripts/notificationsetting.js
+++ b/dashboard-ui/scripts/notificationsetting.js
@@ -4,17 +4,15 @@
 
     function fillItems(elem, items, cssClass, idPrefix, currentList, isEnabledList) {
 
-        var html = '<div data-role="controlgroup">';
+        var html = '<div class="paperCheckboxList paperList" style="padding: .5em 1em;">';
 
         html += items.map(function (u) {
 
-            var id = idPrefix + u.Id;
-
             var isChecked = isEnabledList ? currentList.indexOf(u.Id) != -1 : currentList.indexOf(u.Id) == -1;
 
             var checkedHtml = isChecked ? ' checked="checked"' : '';
 
-            return '<label for="' + id + '">' + u.Name + '</label><input class="' + cssClass + '" type="checkbox" data-itemid="' + u.Id + '" id="' + id + '"' + checkedHtml + ' />';
+            return '<paper-checkbox class="' + cssClass + '" type="checkbox" data-itemid="' + u.Id + '"' + checkedHtml + '>' + u.Name + '</paper-checkbox>';
 
         }).join('');
 
@@ -86,7 +84,7 @@
             fillItems($('.sendToUsersList', page), users, 'chkSendTo', 'chkSendTo', notificationConfig.SendToUsers, true);
             fillItems($('.servicesList', page), services, 'chkService', 'chkService', notificationConfig.DisabledServices);
 
-            $('#chkEnabled', page).checked(notificationConfig.Enabled || false).checkboxradio('refresh');
+            $('#chkEnabled', page).checked(notificationConfig.Enabled || false);
 
             $('#txtTitle', page).val(notificationConfig.Title || typeInfo.DefaultTitle);
 
@@ -135,15 +133,21 @@
                 notificationConfig.Title = null;
             }
 
-            notificationConfig.DisabledMonitorUsers = $('.chkMonitor:not(:checked)', page).get().map(function (c) {
+            notificationConfig.DisabledMonitorUsers = $('.chkMonitor', page).get().filter(function (c) {
+                return !c.checked;
+            }).map(function (c) {
                 return c.getAttribute('data-itemid');
             });
 
-            notificationConfig.SendToUsers = $('.chkSendTo:checked', page).get().map(function (c) {
+            notificationConfig.SendToUsers = $('.chkSendTo', page).get().filter(function (c) {
+                return c.checked;
+            }).map(function (c) {
                 return c.getAttribute('data-itemid');
             });
 
-            notificationConfig.DisabledServices = $('.chkService:not(:checked)', page).get().map(function (c) {
+            notificationConfig.DisabledServices = $('.chkService', page).get().filter(function (c) {
+                return !c.checked;
+            }).map(function (c) {
                 return c.getAttribute('data-itemid');
             });