From ac8f58ab7efefc0532cba8f6de76a8936eb17c72 Mon Sep 17 00:00:00 2001
From: Anand Doshi <anand@erpnext.com>
Date: Thu, 8 Jan 2015 00:22:45 +0530
Subject: [PATCH] [minor] on click of sidebar comments, scroll to comments

---
 frappe/public/css/desk.css                     | 3 +++
 frappe/public/js/frappe/form/form_sidebar.html | 2 +-
 frappe/public/js/frappe/form/sidebar.js        | 4 ++++
 frappe/public/less/desk.less                   | 4 ++++
 4 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/frappe/public/css/desk.css b/frappe/public/css/desk.css
index 9fd791e17f..885fb18cd6 100644
--- a/frappe/public/css/desk.css
+++ b/frappe/public/css/desk.css
@@ -387,6 +387,9 @@ em.link-option {
 .sidebar-section h6 .label {
   font-size: 12px;
 }
+.sidebar-section.sidebar-comments {
+  cursor: pointer;
+}
 .user-actions {
   margin-bottom: 15px;
 }
diff --git a/frappe/public/js/frappe/form/form_sidebar.html b/frappe/public/js/frappe/form/form_sidebar.html
index 461f8929f2..50e81543af 100644
--- a/frappe/public/js/frappe/form/form_sidebar.html
+++ b/frappe/public/js/frappe/form/form_sidebar.html
@@ -2,7 +2,7 @@
     <ul class="sidebar-section user-actions list-unstyled hide">
     </ul>
     {% if (!frm.meta.hide_toolbar) { %}
-    <div class="sidebar-section">
+    <div class="sidebar-section sidebar-comments">
         <h6>{%= __("Comments") %}
             <span class="label label-default n-comments"></span></h6>
     </div>
diff --git a/frappe/public/js/frappe/form/sidebar.js b/frappe/public/js/frappe/form/sidebar.js
index 5a5a467995..015c07a784 100644
--- a/frappe/public/js/frappe/form/sidebar.js
+++ b/frappe/public/js/frappe/form/sidebar.js
@@ -2,11 +2,15 @@ frappe.provide("frappe.ui.form");
 frappe.ui.form.Sidebar = Class.extend({
 	init: function(opts) {
 		$.extend(this, opts);
+		var me = this;
 		this.parent.addClass("hidden-xs");
 		this.wrapper = $(frappe.render_template("form_sidebar",
 			{doctype: this.frm.doctype, frm:this.frm}))
 			.appendTo(this.parent);
 		this.$user_actions = this.wrapper.find(".user-actions");
+		this.wrapper.find(".sidebar-section.sidebar-comments").on("click", function() {
+			$(window).scrollTop(me.frm.footer.wrapper.find(".form-comments").offset().top);
+		});
 	},
 	add_user_action: function(label, click) {
 		$('<a>').html(label).appendTo($('<li>')
diff --git a/frappe/public/less/desk.less b/frappe/public/less/desk.less
index a09ed96c3c..401987d80d 100644
--- a/frappe/public/less/desk.less
+++ b/frappe/public/less/desk.less
@@ -465,6 +465,10 @@ em.link-option {
 	font-size: 12px;
 }
 
+.sidebar-section.sidebar-comments {
+	cursor: pointer;
+}
+
 .user-actions {
 	margin-bottom: 15px;
 }