From bf2be7d79c1c369e60020e178d9840e8b024cbd7 Mon Sep 17 00:00:00 2001
From: Prssanna Desai <prssud@gmail.com>
Date: Tue, 23 Jul 2019 11:59:29 +0530
Subject: [PATCH] fix: use html layout and remove title margin

---
 frappe/public/js/frappe/ui/page.html | 6 ++++--
 frappe/public/js/frappe/ui/page.js   | 3 ---
 frappe/public/less/mobile.less       | 5 -----
 3 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/frappe/public/js/frappe/ui/page.html b/frappe/public/js/frappe/ui/page.html
index 0f259d37df..b79b445000 100644
--- a/frappe/public/js/frappe/ui/page.html
+++ b/frappe/public/js/frappe/ui/page.html
@@ -6,8 +6,10 @@
 				<h1 class="flex" style="margin: auto;">
 					<div class="title-image hide hidden-md hidden-lg">
 					</div>
-					<div class="ellipsis title-text"></div>
-					<span class="indicator whitespace-nowrap hide"></span>
+					<div class="flex fill-width">
+						<div class="ellipsis title-text"></div>
+						<span class="indicator whitespace-nowrap hide"></span>
+					</div>
 				</h1>
 			</div>
 			<div class="text-right col-md-5 col-sm-4 col-xs-6 page-actions">
diff --git a/frappe/public/js/frappe/ui/page.js b/frappe/public/js/frappe/ui/page.js
index aa617905f3..0d3d025ddd 100644
--- a/frappe/public/js/frappe/ui/page.js
+++ b/frappe/public/js/frappe/ui/page.js
@@ -140,9 +140,6 @@ frappe.ui.Page = Class.extend({
 
 	set_indicator: function(label, color) {
 		this.clear_indicator().removeClass("hide").html(`<span>${label}</span>`).addClass(color);
-		if(this.$title_area.find('.title-image').is(':visible')) {
-			$(this.indicator).css('left', '70px');
-		}
 	},
 
 	add_action_icon: function(icon, click) {
diff --git a/frappe/public/less/mobile.less b/frappe/public/less/mobile.less
index 30f3d272e7..a4b18dc0bb 100644
--- a/frappe/public/less/mobile.less
+++ b/frappe/public/less/mobile.less
@@ -54,11 +54,6 @@ body {
 			right: 140px;
 			min-width: 200px;
 		}
-
-		.title-text {
-			margin-top: 10px;
-			//margin-left: 10px;
-		}
 	}
 
 	// navbar & breadcrumbs