From e830b5e4accb8b2874df31b6d6d8e06cbf4efa3a Mon Sep 17 00:00:00 2001
From: Faris Ansari <netchamp.faris@gmail.com>
Date: Tue, 19 Oct 2021 19:13:50 +0530
Subject: [PATCH] fix: minor ui fixes

---
 frappe/public/js/print_format_builder/Field.vue       | 11 +++++++++--
 frappe/public/js/print_format_builder/HTMLEditor.vue  |  9 ++++++++-
 .../js/print_format_builder/LetterHeadEditor.vue      |  4 ++--
 frappe/public/js/print_format_builder/PrintFormat.vue |  2 +-
 4 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/frappe/public/js/print_format_builder/Field.vue b/frappe/public/js/print_format_builder/Field.vue
index 2884ea48c5..ca53402083 100644
--- a/frappe/public/js/print_format_builder/Field.vue
+++ b/frappe/public/js/print_format_builder/Field.vue
@@ -45,7 +45,10 @@
 				>
 					Configure columns
 				</button>
-				<button class="btn btn-xs btn-icon" @click="$set(df, 'remove', true)">
+				<button
+					class="btn btn-xs btn-icon"
+					@click="$set(df, 'remove', true)"
+				>
 					<svg class="icon icon-sm">
 						<use xlink:href="#icon-close"></use>
 					</svg>
@@ -141,7 +144,11 @@ export default {
 								let column = this.get_column_to_add(fieldname);
 								if (column) {
 									this.df.table_columns.push(column);
-									this.$set(this.df, "table_columns", this.df.table_columns);
+									this.$set(
+										this.df,
+										"table_columns",
+										this.df.table_columns
+									);
 									dialog.set_value("add_column", "");
 								}
 							}
diff --git a/frappe/public/js/print_format_builder/HTMLEditor.vue b/frappe/public/js/print_format_builder/HTMLEditor.vue
index 495964f950..17024da503 100644
--- a/frappe/public/js/print_format_builder/HTMLEditor.vue
+++ b/frappe/public/js/print_format_builder/HTMLEditor.vue
@@ -1,7 +1,10 @@
 <template>
 	<div class="html-editor">
 		<div class="d-flex justify-content-end">
-			<button class="btn btn-default btn-xs btn-edit" @click="toggle_edit">
+			<button
+				class="btn btn-default btn-xs btn-edit"
+				@click="toggle_edit"
+			>
 				{{ !editing ? buttonLabel : __("Done") }}
 			</button>
 		</div>
@@ -58,4 +61,8 @@ export default {
 	padding: 1rem;
 	margin-bottom: 1rem;
 }
+
+.html-editor:last-child {
+	margin-bottom: 0;
+}
 </style>
diff --git a/frappe/public/js/print_format_builder/LetterHeadEditor.vue b/frappe/public/js/print_format_builder/LetterHeadEditor.vue
index d8db40ce2e..b38544f341 100644
--- a/frappe/public/js/print_format_builder/LetterHeadEditor.vue
+++ b/frappe/public/js/print_format_builder/LetterHeadEditor.vue
@@ -54,10 +54,10 @@
 					{{ __("Change Letter Head") }}
 				</button>
 				<button
-					class="btn btn-default btn-xs btn-edit ml-2"
+					class="ml-2 btn btn-default btn-xs btn-edit"
 					@click="toggle_edit_letterhead"
 				>
-					{{ !$store.edit_letterhead ? __("Edit") : __("Done") }}
+					{{ !$store.edit_letterhead ? __("Edit Letter Head") : __("Done") }}
 				</button>
 			</div>
 		</div>
diff --git a/frappe/public/js/print_format_builder/PrintFormat.vue b/frappe/public/js/print_format_builder/PrintFormat.vue
index 5138a54a38..1857bae47e 100644
--- a/frappe/public/js/print_format_builder/PrintFormat.vue
+++ b/frappe/public/js/print_format_builder/PrintFormat.vue
@@ -31,7 +31,7 @@
 			v-if="letterhead"
 			:value="letterhead.footer"
 			@change="update_letterhead_footer"
-			:button-label="__('Edit LetterHead Footer')"
+			:button-label="__('Edit Letter Head Footer')"
 		/>
 	</div>
 </template>