From 1adbf90d8c51c44035b48d23f55ab8109eb43deb Mon Sep 17 00:00:00 2001
From: khushi8112 <khushirawat23@navgurukul.org>
Date: Tue, 9 Sep 2025 18:58:45 +0530
Subject: [PATCH] fix: css changes in letterhead

---
 .../sales_invoice_print_format.html                       | 4 ++--
 erpnext/fixtures/letter_head.json                         | 8 ++++----
 erpnext/hooks.py                                          | 8 +++++++-
 3 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/erpnext/accounts/print_format/sales_invoice_print_format/sales_invoice_print_format.html b/erpnext/accounts/print_format/sales_invoice_print_format/sales_invoice_print_format.html
index 65bfd93c2be..6a9b5f24564 100644
--- a/erpnext/accounts/print_format/sales_invoice_print_format/sales_invoice_print_format.html
+++ b/erpnext/accounts/print_format/sales_invoice_print_format/sales_invoice_print_format.html
@@ -148,12 +148,12 @@ print_heading_template=None) -%} {% if letter_head and not no_letterhead %}
 				font-style: normal;
 				font-weight: 420;
 				line-height: 21px;
-				padding: 12px;
+				padding: 5px;
 				letter-spacing: 0.14px;
 			}
 		</style>
 
-	<table class="info-table mt-10 mb-20">
+	<table class="info-table mb-20">
 		<tr>
 			<td style="width: 50%">
 				<span class="heading">{{ _("Customer Name") }}:</span> {{doc.customer_name }}
diff --git a/erpnext/fixtures/letter_head.json b/erpnext/fixtures/letter_head.json
index 4b9e7d9cc46..b956a8c64e3 100644
--- a/erpnext/fixtures/letter_head.json
+++ b/erpnext/fixtures/letter_head.json
@@ -1,7 +1,7 @@
 [
  {
   "align": "Left",
-  "content": "<style>\n     .logo-address {\n        display: flex;\n        align-items: center;\n        gap: 15px;\n        width: 50%;\n    }\n    \n    .logo {\n        border: 1px solid #EDEDED;\n        border-radius: 15px;\n    }\n    \n    .company-name {\n        color: var(--ink-gray-8, #171717);\n        font-size: 15px;\n        font-style: normal;\n        font-weight: 600;\n        line-height: 23px;\n        letter-spacing: 0.2px;\n    }\n\n    .address{\n        display: flex;\n        flex-direction: column;\n        justify-content: center;\n        align-items: flex-start;\n    }\n    \n    .letterhead-right {\n        display: flex;\n        flex-direction: column;\n        align-items: flex-start;\n        gap: 2px;\n    }\n    \n    .letterhead-container {\n      display: flex;\n      padding: 12px 0;\n      justify-content: space-between;\n      margin-bottom: 30px;\n    }\n    \n    \n    .letterhead-right-div {\n        display: flex;\n        align-items: flex-start;\n        gap: 30px;\n    }\n    \n    .letterhead-title {\n        color: #7C7C7C;\n        width: 60px;\n        flex: 1 0 0;\n    }\n    \n\n</style>\n\n<div class=\"letterhead-container\">\n    <div class=\"logo-address\">\n      <div class=\"logo\">\n        {% if frappe.db.get_value(\"Company\", doc.company, \"logo_for_printing\") %}\n          <img src=\"{{ frappe.utils.get_url(frappe.db.get_value('Company', doc.company, 'logo_for_printing')) }}\" style=\"width: 90px height: 90px;\">\n        {% endif %}\n      </div>\n      <div class=\"address\">\n        <div class=\"company-name\">{{ doc.company }}</div>\n        {% set company_address = frappe.get_all(\n            \"Dynamic Link\",\n            fields=[\"parent\"],\n            filters={\n              \"link_doctype\": \"Company\",\n              \"link_name\": doc.company,\n              \"parenttype\": \"Address\"\n            },\n            limit=1\n        ) %}\n    \n        {% if company_address %}\n          {% set address = frappe.get_doc(\"Address\", company_address[0].parent) %}\n          {{ address.address_line1 or \"\" }}<br>\n          {% if address.address_line2 %}{{ address.address_line2 }}<br>{% endif %}\n          {{ address.city or \"\" }} {{ address.state or \"\" }} {{ address.pincode or \"\" }} {{ address.country or \"\" }}<br>\n        {% endif %}\n      </div>\n    </div>\n\n  <div class=\"letterhead-right\">\n    {% set website = frappe.db.get_value(\"Company\", doc.company, \"website\") %}\n    {% set email = frappe.db.get_value(\"Company\", doc.company, \"email\") %}\n    {% set phone_no = frappe.db.get_value(\"Company\", doc.company, \"phone_no\") %}\n    {% set pan = frappe.db.get_value(\"Company\", doc.company, \"pan\") %}\n\n    {% if website %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"Website:\") }}</div>\n            <div>{{ website }}</div><br>\n        </div>\n    {% endif %}\n    {% if email %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"Email:\") }}</div>\n            <div>{{ email }}</div><br>\n        </div>\n    {% endif %}\n    {% if phone_no %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"Contact:\") }}</div>\n            <div>{{ phone_no }}</div><br>\n        </div>\n    {% endif %}\n    {% if pan %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"PAN:\") }}</div>\n            <div>{{ pan }}</div><br>\n        </div>\n    {% endif %}\n  </div>\n</div>\n",
+  "content": "<style>\n     .logo-address {\n        display: flex;\n        align-items: center;\n        gap: 15px;\n        width: 50%;\n    }\n    \n    .logo {\n        border: 1px solid #EDEDED;\n        border-radius: 15px;\n    }\n    \n    .company-name {\n        color: var(--ink-gray-8, #171717);\n        font-size: 15px;\n        font-style: normal;\n        font-weight: 600;\n        line-height: 23px;\n        letter-spacing: 0.2px;\n    }\n\n    .address{\n        display: flex;\n        flex-direction: column;\n        justify-content: center;\n        align-items: flex-start;\n    }\n    \n    .letterhead-right {\n        display: flex;\n        flex-direction: column;\n        align-items: flex-start;\n        gap: 2px;\n    }\n    \n    .letterhead-container {\n      display: flex;\n      padding: 12px 0 0;\n      justify-content: space-between;\n      margin-bottom: 30px;\n    }\n    \n    \n    .letterhead-right-div {\n        display: flex;\n        align-items: flex-start;\n        gap: 0 10px;\n    }\n    \n    .letterhead-title {\n        color: #7C7C7C;\n        width: 60px;\n        flex: 1 0 0;\n    }\n    \n\n</style>\n\n<div class=\"letterhead-container\">\n    <div class=\"logo-address\">\n      <div class=\"logo\">\n        {% if frappe.db.get_value(\"Company\", doc.company, \"company_logo\") %}\n          <img src=\"{{ frappe.utils.get_url(frappe.db.get_value('Company', doc.company, 'company_logo')) }}\" style=\"width: 90px height: 90px;\">\n        {% endif %}\n      </div>\n      <div class=\"address\">\n        <div class=\"company-name\">{{ doc.company }}</div>\n        {% set company_address = frappe.get_all(\n            \"Dynamic Link\",\n            fields=[\"parent\"],\n            filters={\n              \"link_doctype\": \"Company\",\n              \"link_name\": doc.company,\n              \"parenttype\": \"Address\"\n            },\n            limit=1\n        ) %}\n    \n        {% if company_address %}\n          {% set address = frappe.get_doc(\"Address\", company_address[0].parent) %}\n          {{ address.address_line1 or \"\" }}<br>\n          {% if address.address_line2 %}{{ address.address_line2 }}<br>{% endif %}\n          {{ address.city or \"\" }} {{ address.state or \"\" }} {{ address.pincode or \"\" }} {{ address.country or \"\" }}<br>\n        {% endif %}\n      </div>\n    </div>\n\n  <div class=\"letterhead-right\">\n    {% set website = frappe.db.get_value(\"Company\", doc.company, \"website\") %}\n    {% set email = frappe.db.get_value(\"Company\", doc.company, \"email\") %}\n    {% set phone_no = frappe.db.get_value(\"Company\", doc.company, \"phone_no\") %}\n    {% set pan = frappe.db.get_value(\"Company\", doc.company, \"pan\") %}\n\n    {% if website %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"Website:\") }}</div>\n            <div>{{ website }}</div><br>\n        </div>\n    {% endif %}\n    {% if email %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"Email:\") }}</div>\n            <div>{{ email }}</div><br>\n        </div>\n    {% endif %}\n    {% if phone_no %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"Contact:\") }}</div>\n            <div>{{ phone_no }}</div><br>\n        </div>\n    {% endif %}\n    {% if pan %}\n        <div class=\"letterhead-right-div\">\n            <div class=\"letterhead-title\">{{ _(\"PAN:\") }}</div>\n            <div>{{ pan }}</div><br>\n        </div>\n    {% endif %}\n  </div>\n</div>\n",
   "disabled": 0,
   "docstatus": 0,
   "doctype": "Letter Head",
@@ -18,13 +18,13 @@
   "image_width": 0.0,
   "is_default": 0,
   "letter_head_name": "Sales Invoice Letter Head",
-  "modified": "2025-09-09 14:31:07.209406",
+  "modified": "2025-09-09 18:25:09.704061",
   "name": "Sales Invoice Letter Head",
   "source": "HTML"
  },
  {
   "align": "Left",
-  "content": "<style>\n    .letterhead-container {\n      padding: 12px;\n      margin-bottom: 30px;\n      border-radius: 18px;\n      background: var(--surface-gray-1, #F8F8F8);\n    }\n    \n    .logo-container {\n        display: flex;\n        align-items: flex-start;\n        justify-content: space-between;\n        margin-right: 2px;\n        margin-bottom: 10px;\n    }\n\n    .logo {\n        border: 1px solid #EDEDED;\n        border-radius: 15px;\n        width: 90px;\n        height: 90px;\n    }\n    \n    .invoice {\n        text-align: right;\n    }\n\n    .company-details {\n        display: flex;\n        align-items: flex-start;\n        justify-content: space-between;\n        flex-wrap: wrap;\n    }\n    \n    .address{\n        display: flex;\n        flex-direction: column;\n        justify-content: center;\n        align-items: flex-start;\n        width: 50%;\n    }\n    \n    .company-name {\n        color: var(--ink-gray-8, #171717);\n        font-size: 15px;\n        font-style: normal;\n        font-weight: 600;\n        line-height: 23px;\n        letter-spacing: 0.2px;\n    }\n\n    \n    .contact-details {\n        display: flex;\n        flex-direction: column;\n        align-items: flex-start;\n        gap: 2px;\n    }\n    \n    .letterhead-right-div {\n        display: flex;\n        align-items: flex-start;\n        gap: 30px;\n    }\n    \n    .letterhead-title {\n        color: #7C7C7C;\n        width: 60px;\n        flex: 1 0 0;\n    }\n    \n</style>\n\n<div class=\"letterhead-container\">\n    <div class=\"logo-container\">\n        <div class=\"logo\">\n            {% if frappe.db.get_value(\"Company\", doc.company, \"logo_for_printing\") %}\n              <img src=\"{{ frappe.utils.get_url(frappe.db.get_value('Company', doc.company, 'logo_for_printing')) }}\" style=\"width: 90px height: 90px;\">\n            {% endif %}\n        </div>\n        <div class=\"invoice\">\n            <div>\n                {{ _(\"Sales Invoice\") }}\n            </div>\n            <div>\n                {{ doc.name }}\n            </div>\n        </div>\n    </div>\n    <div class=\"company-details\">\n        <div class=\"address\">\n            <div class=\"company-name\">{{ doc.company }}</div>\n                {% set company_address = frappe.get_all(\n                    \"Dynamic Link\",\n                    fields=[\"parent\"],\n                    filters={\n                      \"link_doctype\": \"Company\",\n                      \"link_name\": doc.company,\n                      \"parenttype\": \"Address\"\n                    },\n                    limit=1\n                ) %}\n            \n                {% if company_address %}\n                  {% set address = frappe.get_doc(\"Address\", company_address[0].parent) %}\n                  {{ address.address_line1 or \"\" }}<br>\n                  {% if address.address_line2 %}{{ address.address_line2 }}<br>{% endif %}\n                  {{ address.city or \"\" }} {{ address.state or \"\" }} {{ address.pincode or \"\" }} {{ address.country or \"\" }}<br>\n                {% endif %}\n        </div>\n\n        <div class=\"contact-details\">\n            {% set website = frappe.db.get_value(\"Company\", doc.company, \"website\") %}\n            {% set email = frappe.db.get_value(\"Company\", doc.company, \"email\") %}\n            {% set phone_no = frappe.db.get_value(\"Company\", doc.company, \"phone_no\") %}\n            {% set pan = frappe.db.get_value(\"Company\", doc.company, \"pan\") %}\n        \n            {% if website %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"Website:\") }}</div>\n                    <div>{{ website }}</div><br>\n                </div>\n            {% endif %}\n            {% if email %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"Email:\") }}</div>\n                    <div>{{ email }}</div><br>\n                </div>\n            {% endif %}\n            {% if phone_no %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"Contact:\") }}</div>\n                    <div>{{ phone_no }}</div><br>\n                </div>\n            {% endif %}\n            {% if pan %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"PAN:\") }}</div>\n                    <div>{{ pan }}</div><br>\n                </div>\n            {% endif %}\n        </div>\n    </div>\n</div>\n",
+  "content": "<style>\n    .letterhead-container {\n      padding: 12px;\n      margin-bottom: 30px;\n      border-radius: 18px;\n      background: var(--surface-gray-1, #F8F8F8);\n    }\n    \n    .logo-container {\n        display: flex;\n        align-items: flex-start;\n        justify-content: space-between;\n        margin-right: 2px;\n        margin-bottom: 10px;\n    }\n\n    .logo {\n        border: 1px solid #EDEDED;\n        border-radius: 15px;\n        width: 75px;\n    }\n    \n    .invoice {\n        text-align: right;\n    }\n\n    .company-details {\n        display: flex;\n        align-items: flex-start;\n        justify-content: space-between;\n        flex-wrap: wrap;\n    }\n    \n    .address{\n        display: flex;\n        flex-direction: column;\n        justify-content: center;\n        align-items: flex-start;\n        width: 50%;\n    }\n    \n    .company-name {\n        color: var(--ink-gray-8, #171717);\n        font-size: 15px;\n        font-style: normal;\n        font-weight: 600;\n        line-height: 23px;\n        letter-spacing: 0.2px;\n    }\n\n    \n    .contact-details {\n        display: flex;\n        flex-direction: column;\n        align-items: flex-start;\n        gap: 2px;\n    }\n    \n    .letterhead-right-div {\n        display: flex;\n        align-items: flex-start;\n        gap: 0 10px;\n    }\n    \n    .letterhead-title {\n        color: #7C7C7C;\n        width: 60px;\n        flex: 1 0 0;\n    }\n    \n</style>\n\n<div class=\"letterhead-container\">\n    <div class=\"logo-container\">\n        <div class=\"logo\">\n            {% if frappe.db.get_value(\"Company\", doc.company, \"company_logo\") %}\n              <img src=\"{{ frappe.utils.get_url(frappe.db.get_value('Company', doc.company, 'company_logo')) }}\">\n            {% endif %}\n        </div>\n        <div class=\"invoice\">\n            <div style=\"font-size: 16px; font-weight: bold;\">\n                {{ _(\"Sales Invoice\") }}\n            </div>\n            <div style=\"color:#7C7C7C;\">\n                {{ doc.name }}\n            </div>\n        </div>\n    </div>\n    <div class=\"company-details\">\n        <div class=\"address\">\n            <div class=\"company-name\">{{ doc.company }}</div>\n                <span class=\"font-size: 13px;\">\n                    {% set company_address = frappe.get_all(\n                        \"Dynamic Link\",\n                        fields=[\"parent\"],\n                        filters={\n                          \"link_doctype\": \"Company\",\n                          \"link_name\": doc.company,\n                          \"parenttype\": \"Address\"\n                        },\n                        limit=1\n                    ) %}\n                \n                    {% if company_address %}\n                      {% set address = frappe.get_doc(\"Address\", company_address[0].parent) %}\n                      {{ address.address_line1 or \"\" }}<br>\n                      {% if address.address_line2 %}{{ address.address_line2 }}<br>{% endif %}\n                      {{ address.city or \"\" }} {{ address.state or \"\" }} {{ address.pincode or \"\" }} {{ address.country or \"\" }}<br>\n                    {% endif %}\n                </span>\n        </div>\n\n        <div class=\"contact-details\">\n            {% set website = frappe.db.get_value(\"Company\", doc.company, \"website\") %}\n            {% set email = frappe.db.get_value(\"Company\", doc.company, \"email\") %}\n            {% set phone_no = frappe.db.get_value(\"Company\", doc.company, \"phone_no\") %}\n            {% set pan = frappe.db.get_value(\"Company\", doc.company, \"pan\") %}\n        \n            {% if website %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"Website:\") }}</div>\n                    <div>{{ website }}</div><br>\n                </div>\n            {% endif %}\n            {% if email %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"Email:\") }}</div>\n                    <div>{{ email }}</div><br>\n                </div>\n            {% endif %}\n            {% if phone_no %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"Contact:\") }}</div>\n                    <div>{{ phone_no }}</div><br>\n                </div>\n            {% endif %}\n            {% if pan %}\n                <div class=\"letterhead-right-div\">\n                    <div class=\"letterhead-title\">{{ _(\"PAN:\") }}</div>\n                    <div>{{ pan }}</div><br>\n                </div>\n            {% endif %}\n        </div>\n    </div>\n</div>\n",
   "disabled": 0,
   "docstatus": 0,
   "doctype": "Letter Head",
@@ -41,7 +41,7 @@
   "image_width": 0.0,
   "is_default": 1,
   "letter_head_name": "Print Format Letter Head",
-  "modified": "2025-09-09 14:31:07.204974",
+  "modified": "2025-09-09 18:11:28.309162",
   "name": "Print Format Letter Head",
   "source": "HTML"
  }
diff --git a/erpnext/hooks.py b/erpnext/hooks.py
index 540fd6c36c4..213efd9a0f6 100644
--- a/erpnext/hooks.py
+++ b/erpnext/hooks.py
@@ -600,7 +600,13 @@ user_privacy_documents = [
 	},
 ]
 
-fixtures = ["Letter Head"]
+fixtures = [
+	{
+		"dt": "Letter Head",
+		"filters": [["name", "in", ["Sales Invoice Letter Head", "Print Format Letter Head"]]],
+	}
+]
+
 
 # ERPNext doctypes for Global Search
 global_search_doctypes = {