[fix] open filtered list when clicked on notifications circle

frappe/core/page/desktop/desktop.js

@@ -37,6 +37,7 @@ $.extend(frappe.desktop, {
 			label: 'Explore',
 			_label: __('Explore'),
 			_id: 'Explore',
+			_doctype: '',
 			icon: 'octicon octicon-telescope',
 			color: '#7578f6',
 			link: 'modules'
@@ -71,6 +72,12 @@ $.extend(frappe.desktop, {
 				frappe.desktop.open_module($(this).parent());
 			});
 		}
+		frappe.desktop.wrapper.on("click", ".circle", function() {
+			var doctype = $(this).attr('data-doctype');
+			if(doctype) {
+				frappe.set_route('List', doctype, frappe.ui.notifications.get_filters(doctype));
+			}
+		});
 	},
 
 	open_module: function(parent) {
@@ -122,7 +129,6 @@ $.extend(frappe.desktop, {
 	},
 
 	show_pending_notifications: function() {
-
 		if (!frappe.boot.notification_info.module_doctypes) {
 			return;
 		}

frappe/core/page/desktop/desktop_module_icon.html

@@ -2,7 +2,7 @@
 	data-name="{{ module_name }}" data-link="{{ link }}" title="{{ _label }}">
 	{{ app_icon }}
 	<div class="case-label text-ellipsis">
-		<div class="circle module-count-{{ _id }}" style="display: none;">
+		<div class="circle module-count-{{ _id }}" data-doctype="{{ _doctype }}" style="display: none;">
 			<span class="circle-text"></span>
 		</div>
 		<!-- <span id="module-count-{{ _id }}" class="octicon octicon-primitive-dot circle" style="display: None"></span> -->

frappe/desk/page/modules/modules.js

@@ -72,19 +72,20 @@ frappe.pages['modules'].on_page_load = function(wrapper) {
 		//setup_section_toggle();
 		frappe.app.update_notification_count_in_modules();
 
-		page.main.find('.module-section-link').on("click", function(event) {
-			// if clicked on open notification!
-			if (event.target.classList.contains("open-notification")) {
-				var doctype = event.target.getAttribute("data-doctype");
-				frappe.route_options = frappe.boot.notification_info.conditions[doctype];
-				return false;
-			}
-			if($(this).attr("data-type")==="help") {
-				frappe.help.show_video($(this).attr("data-youtube-id"));
-				return false;
+		// notifications click
+		page.main.on("click", '.open-notification', function(event) {
+			var doctype = $(this).attr('data-doctype');
+			if(doctype) {
+				frappe.set_route('List', doctype, frappe.ui.notifications.get_filters(doctype));
 			}
 		});
 
+		// help click
+		page.main.on("click", '.module-section-link[data-type="help"]', function(event) {
+			frappe.help.show_video($(this).attr("data-youtube-id"));
+			return false;
+		});
+
 	}
 
 	var process_data = function(data) {

frappe/desk/page/modules/modules_section.html

@@ -10,7 +10,7 @@
 			var item = section.items[j];
 			if(item.shown) { %}
 				<div style="min-height: 22px; margin-top: 8px;">
-					<a class="module-section-link" data-type="{{ item.type }}"
+					<a class="module-section-link small" data-type="{{ item.type }}"
 						{% if(item.type==="help") { %}
 							data-youtube-id="{{ item.youtube_id }}"{% } %}
 						href="#{{ item.route }}" style="{{ item.style }}">

frappe/model/base_document.py

@@ -356,6 +356,8 @@ class BaseDocument(object):
 		frappe.db.set_value(self.doctype, self.name, fieldname, value,
 			self.modified, self.modified_by, update_modified=update_modified)
 
+		self.run_trigger('on_change')
+
 	def _fix_numeric_types(self):
 		for df in self.meta.get("fields"):
 			if df.fieldtype == "Check":

frappe/public/css/desk.css

@@ -448,6 +448,7 @@ textarea.form-control {
   padding: 0 8px;
   color: #fff;
   border-radius: 10px;
+  cursor: pointer;
 }
 /* on small screens, show only icons on top */
 @media (max-width: 767px) {

frappe/public/css/desktop.css

@@ -78,6 +78,7 @@ body[data-route="desktop"] .navbar-default {
   text-align: center;
   text-shadow: none;
   letter-spacing: normal;
+  cursor: pointer;
 }
 .app-icon:hover path {
   fill: #fff;

frappe/public/css/module.css

@@ -18,7 +18,6 @@
 }
 .module-section .module-section-link {
   line-height: 1.5em;
-  font-size: 14px;
 }
 .module-section-column {
   padding: 30px;

frappe/public/js/frappe/desk.js

@@ -370,6 +370,10 @@ frappe.get_module = function(m, default_module) {
 		module._label = __(module.label);
 	}
 
+	if(!module._doctype) {
+		module._doctype = '';
+	}
+
 	module._setup = true;
 
 	return module;

frappe/public/js/frappe/ui/toolbar/notifications.js

@@ -103,10 +103,10 @@ frappe.ui.notifications.config = {
 
 frappe.views.show_open_count_list = function(element) {
 	var doctype = $(element).attr("data-doctype");
-	var condition = frappe.boot.notification_info.conditions[doctype];
+	var filters = frappe.ui.notifications.get_filters(doctype);
 
-	if(condition && $.isPlainObject(condition)) {
-		frappe.route_options = condition;
+	if(filters) {
+		frappe.route_options = filters;
 	}
 
 	var route = frappe.get_route();
@@ -116,3 +116,11 @@ frappe.views.show_open_count_list = function(element) {
 		frappe.set_route("List", doctype);
 	}
 }
+
+frappe.ui.notifications.get_filters = function(doctype) {
+	var conditions = frappe.boot.notification_info.conditions[doctype];
+
+	if(conditions && $.isPlainObject(conditions)) {
+		return conditions;
+	}
+}

frappe/public/less/desk.less

@@ -286,6 +286,7 @@ textarea.form-control {
     padding:0 8px;
     color:#fff;
     border-radius:10px;
+	cursor: pointer;
 }
 
 /* on small screens, show only icons on top */

frappe/public/less/desktop.less

@@ -90,6 +90,7 @@ body[data-route=""] .navbar-default, body[data-route="desktop"] .navbar-default
 	text-align: center;
 	text-shadow: none;
 	letter-spacing: normal;
+	cursor: pointer;
 }
 
 .app-icon:hover path {

frappe/public/less/module.less

@@ -24,7 +24,7 @@
 
 	.module-section-link {
 		line-height: 1.5em;
-		font-size: 14px;
+		// font-size: 14px;
 	}
 }