From de0012e1f6d05d3da436de6f0ddf8d3a99565c8a Mon Sep 17 00:00:00 2001
From: Rushabh Mehta <rmehta@gmail.com>
Date: Wed, 24 Apr 2013 14:39:54 +0530
Subject: [PATCH] [form] style fixes

---
 public/css/legacy/fields.css                 |  4 ++--
 public/js/legacy/widgets/form/fields.js      |  5 +++--
 public/js/legacy/widgets/form/form_fields.js |  3 ---
 public/js/wn/form/grid.js                    | 12 ++++++------
 public/js/wn/form/layout.js                  |  6 ++++--
 5 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/public/css/legacy/fields.css b/public/css/legacy/fields.css
index e0f981d01f..715f884040 100644
--- a/public/css/legacy/fields.css
+++ b/public/css/legacy/fields.css
@@ -21,10 +21,10 @@
 
 .disp-area { 
 	background-color: #eee;
-	padding: 3px;
+	padding: 6px;
 	margin-bottom: 7px;
 	border-radius: 3px;	
 }
 .disp-area-no-val { 
-	min-height: 17px; 
+	height: 30px; 
 }
\ No newline at end of file
diff --git a/public/js/legacy/widgets/form/fields.js b/public/js/legacy/widgets/form/fields.js
index a66b791008..4f6cac7d3d 100644
--- a/public/js/legacy/widgets/form/fields.js
+++ b/public/js/legacy/widgets/form/fields.js
@@ -45,13 +45,14 @@ Field.prototype.make_body = function() {
 	this.$wrapper = $('<div class="control-group">\
 		<label class="control-label"></label>\
 		<div class="controls">\
+			<div class="control-input"></div>\
 			<div class="control-value"></div>\
 		</div>\
 	</div>').appendTo(this.parent);
 	this.wrapper = this.$wrapper.get(0);
 	
 	this.label_area = this.label_span = this.$wrapper.find(".control-label").get(0);
-	this.input_area = this.$wrapper.find(".controls").get(0);
+	this.input_area = this.$wrapper.find(".control-input").get(0);
 	this.disp_area = this.$wrapper.find(".control-value").get(0);
 
 	// set description
@@ -449,7 +450,7 @@ LinkField.prototype.make_input = function() {
 		this.input = this.txt;	
 	} else {
 		me.input = me.input_area;
-		me.input_group = $('<div class="input-group link-field col-span-12">').appendTo(me.input_area);
+		me.input_group = $('<div class="input-group link-field">').appendTo(me.input_area);
 			
 		me.txt = $('<input type="text" style="margin-right: 0px;">')
 			.appendTo(me.input_group).get(0);
diff --git a/public/js/legacy/widgets/form/form_fields.js b/public/js/legacy/widgets/form/form_fields.js
index 13c101a4d9..5236060d94 100644
--- a/public/js/legacy/widgets/form/form_fields.js
+++ b/public/js/legacy/widgets/form/form_fields.js
@@ -145,6 +145,3 @@ _f.CodeField.prototype.set_disp = function(val) {
 			+val+'</textarea>';
 	}
 }
-
-// ======================================================================================
-
diff --git a/public/js/wn/form/grid.js b/public/js/wn/form/grid.js
index e08d75246f..cdcb1557ea 100644
--- a/public/js/wn/form/grid.js
+++ b/public/js/wn/form/grid.js
@@ -149,12 +149,7 @@ wn.ui.form.GridRow = Class.extend({
 	},
 	set_button_events: function() {
 		var me = this;
-		
-		if(this.grid.display_status!="Write") {
-			this.wrapper.find(".btn-danger, .grid-insert-row").toggle(false);
-			return;
-		}
-		
+				
 		this.wrapper.find(".btn-danger").click(function() {
 			me.wrapper.fadeOut(function() {
 				wn.model.clear_doc(me.doc.doctype, me.doc.name);
@@ -252,6 +247,11 @@ wn.ui.form.GridRow = Class.extend({
 				cnt++;
 			}
 		});
+
+		if(this.grid.display_status!="Write") {
+			this.wrapper.find(".btn-danger, .grid-insert-row").toggle(false);
+			return;
+		}		
 	},
 	set_data: function() {
 		this.wrapper.data({
diff --git a/public/js/wn/form/layout.js b/public/js/wn/form/layout.js
index 6308672362..02bf47c0af 100644
--- a/public/js/wn/form/layout.js
+++ b/public/js/wn/form/layout.js
@@ -43,9 +43,11 @@ wn.ui.form.Layout = Class.extend({
 	},
 	make_column: function(df) {
 		this.column = $('<div class="form-column">\
-			<form class="form-horizontal" action="javascript:false;">\
+			<form class="form-horizontal">\
 			</form>\
-		</div>').appendTo(this.section).find("form");
+		</div>').appendTo(this.section)
+			.find("form")
+			.on("submit", function() { return false; });
 		
 		// distribute all columns equally
 		var colspan = cint(12 / this.section.find(".form-column").length);