grid, reportview fix webnotes/wnframework#293 webnotes/erpnext#1300

webnotes/public/js/lib/slickgrid/slick-default-theme.css

@@ -71,10 +71,6 @@ classes should alter those!
   background: white;
 }
 
-.slick-cell.selected {
-  background-color: beige;
-}
-
 .slick-cell.active {
   border-color: gray;
   border-style: solid;
@@ -92,6 +88,10 @@ classes should alter those!
   background-color: #f9f9f9;
 }
 
+.slick-cell.selected {
+  background-color: beige !important;
+}
+
 .slick-row.ui-state-active {
   background: #F5F7D7;
 }

webnotes/public/js/wn/form/control.js

@@ -157,7 +157,9 @@ wn.ui.form.ControlInput = wn.ui.form.Control.extend({
 		} else {
 			this.label_area = this.label_span = this.$wrapper.find("label").get(0);
 			this.input_area = this.$wrapper.find(".control-input").get(0);
-			this.disp_area = this.$wrapper.find(".control-value").get(0);	
+			// keep a separate display area to rendered formatted values
+			// like links, currencies, HTMLs etc.
+			this.disp_area = this.$wrapper.find(".control-value").get(0);
 		}
 	},
 	set_max_width: function() {

webnotes/public/js/wn/form/grid.js

@@ -17,7 +17,7 @@ wn.ui.form.Grid = Class.extend({
 				<div class="rows"></div>\
 				<div class="small">\
 					<a href="#" class="grid-add-row pull-right">+ '+wn._("Add new row")+'.</a>\
-					<span class="text-muted pull-right" style="margin-right: 5px;">' + wn._("Click on row to edit.") + '</span>\
+					<span class="text-muted pull-right" style="margin-right: 5px;">' + wn._("Click on row to view / edit.") + '</span>\
 					<div class="clearfix"></div>\
 				</div>\
 			</div>\

webnotes/public/js/wn/views/reportview.js

@@ -208,6 +208,7 @@ wn.views.ReportView = wn.ui.Listing.extend({
 				var docfield = wn.model.get_std_field(c[0]);
 				if(c[0]=="name") { 
 					docfield.options = me.doctype;
+					docfield.parent = me.doctype;
 				}
 			}
 			coldef = {
@@ -285,6 +286,20 @@ wn.views.ReportView = wn.ui.Listing.extend({
 			me.grid.invalidateRows(args.rows);
 			me.grid.render();
 		});
+
+		this.grid.onHeaderClick.subscribe(function(e, args) {
+			var df = args.column.docfield,
+				sort_by = df.parent + "." + df.fieldname;
+							
+			if(sort_by===me.sort_by_select.val()) {
+				me.sort_order_select.val(me.sort_order_select.val()==="asc" ? "desc" : "asc");
+			} else {
+				me.sort_by_select.val(df.parent + "." + df.fieldname);
+				me.sort_order_select.val("asc");
+			}
+
+			me.run();
+		});
 	},
 	
 	edit_cell: function(row, docfield) {