fix: Remove scrolling on focusout event for touchscreen devices

frappe · Nov 6, 2020 · 59c5437 · 59c5437
1 parent ca78636
commit 59c5437
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 13 deletions.
diff --git a/frappe/public/js/frappe/form/controls/base_input.js b/frappe/public/js/frappe/form/controls/base_input.js
@@ -133,18 +133,6 @@ frappe.ui.form.ControlInput = frappe.ui.form.Control.extend({
 			me.parse_validate_and_set_in_model(me.get_input_value(), e);
 		});
 	},
-	bind_focusout: function() {
-		// on touchscreen devices, scroll to top
-		// so that static navbar and page head don't overlap the input
-		if (frappe.dom.is_touchscreen()) {
-			var me = this;
-			this.$input && this.$input.on("focusout", function() {
-				if (frappe.dom.is_touchscreen()) {
-					frappe.utils.scroll_to(me.$wrapper);
-				}
-			});
-		}
-	},
 	set_label: function(label) {
 		if(label) this.df.label = label;
 

diff --git a/frappe/public/js/frappe/form/controls/data.js b/frappe/public/js/frappe/form/controls/data.js
@@ -21,7 +21,6 @@ frappe.ui.form.ControlData = frappe.ui.form.ControlInput.extend({
 		this.input = this.$input.get(0);
 		this.has_input = true;
 		this.bind_change_event();
-		this.bind_focusout();
 		this.setup_autoname_check();
 		if (this.df.options == 'Phone') {
 			this.setup_phone();