From 754ae4be5e9bfeca2123b2a79e308ca217ddc4b4 Mon Sep 17 00:00:00 2001
From: Alexander Lutay <alutay@sipwise.com>
Date: Fri, 19 Feb 2021 19:30:34 +0100
Subject: [PATCH] TT#103252 Fix 'Panel Branding': codestyle, layout,
 localization

* Keep all buttons in the same "line".
* Add localization for all strings on the page.
* Align the code accordingly to the Sipwise code style.

Change-Id: I60df82b0a9f75b353ccf30d9ad5a7a94d05f46f4
---
 share/templates/reseller/branding.tt | 68 ++++++++++++----------------
 1 file changed, 30 insertions(+), 38 deletions(-)

diff --git a/share/templates/reseller/branding.tt b/share/templates/reseller/branding.tt
index 14cfd8db09..34de5647cd 100644
--- a/share/templates/reseller/branding.tt
+++ b/share/templates/reseller/branding.tt
@@ -1,14 +1,18 @@
 [% site_config.title = c.loc('Panel Branding') -%]
 
-<div class="row">
-    <span>
-        <a class="btn btn-primary btn-large" href="[% c.uri_for('/back') %]"><i class="icon-arrow-left"></i> [% c.loc('Back') %]</a>
-    </span>
-</div>
-[% back_created = 1 -%]
-
 <div class="ngcp-separator"></div>
 
+<span>
+    <a href="[% c.uri_for('/back') %]" class="btn btn-primary btn-large"><i class="icon-arrow-left"></i> [% c.loc('Back') %]</a>
+    [% back_created = 1 -%]
+    [% UNLESS c.user.read_only -%]
+        <a href="[% c.uri_for_action('/reseller/edit_branding_css', c.req.captures) %]" class="btn btn-large btn-primary"><i class="icon-edit"></i> [% c.loc('Edit Branding') %]</a>
+        [% IF branding.logo -%]
+            <a href="[% c.uri_for_action('/reseller/delete_branding_logo', c.req.captures) %]" class="btn btn-large btn-primary"><i class="icon-trash"></i> [% c.loc('Delete Logo') %]</a>
+        [% END -%]
+    [% END -%]
+</span>
+
 <div class="row">
     [% FOREACH m IN messages -%]
         <div class="alert alert-[% m.type %]">[% m.text %]</div>
@@ -16,47 +20,35 @@
     [% IF reseller.first.status != "active" -%]
         <div class="alert">[% c.loc('Reseller is <b>[_1]</b>', reseller.first.status) %]</div>
     [% END -%]
-[% messages = [] -%]
+    [% messages = [] -%]
 </div>
 
-
-            [% UNLESS c.user.read_only -%]
-            <span>
-                <a href="[% c.uri_for_action('/reseller/edit_branding_css', c.req.captures) %]" class="btn btn-large btn-primary"><i class="icon-edit"></i> Edit Branding</a>
-                [% IF branding.logo -%]
-                <a href="[% c.uri_for_action('/reseller/delete_branding_logo', c.req.captures) %]" class="btn btn-large btn-primary"><i class="icon-trash"></i> Delete Logo</a>
-                [% END -%]
-            </span>
-            <div class="ngcp-separator"></div>
-            [% END -%]
-
-            [% IF branding.defined -%]
-                [% IF branding.logo.defined -%]
-                    <h3>Custom Logo</h3>
-                    <img src="[% c.uri_for_action('/reseller/get_branding_logo', c.req.captures) %]">
-                    <p>
-                        [% c.loc("You can use the logo by adding the following CSS to the Custom CSS below:") %]
+[% IF branding.defined -%]
+    [% IF branding.logo.defined -%]
+        <h3>[% c.loc('Custom Logo') %]</h3>
+        <img src="[% c.uri_for_action('/reseller/get_branding_logo', c.req.captures) %]">
+        <p>
+            [% c.loc("You can use the logo by adding the following CSS to the Custom CSS below:") %]
 <pre>
 #header .brand {
     background: url([% c.uri_for_action('/reseller/get_branding_logo', c.req.captures) %]) no-repeat 0 0;
     background-size: 280px 32px;
 }
 </pre>
-                    </p>
-                [% ELSE -%]
-                    No logo uploaded.
-                [% END -%]
-
+        </p>
+    [% ELSE -%]
+        [% c.loc('No logo uploaded') %]
+    [% END -%]
 
-                <h3>Custom CSS</h3>
-                <p>
-                <pre style="color:#000">
+    <h3>[% c.loc('Custom CSS') %]</h3>
+    <p>
+    <pre style="color:#000">
 [% branding.css %]
-                </pre>
-                </p>
-            [% ELSE -%]
-                No branding specified, using standard branding.
-            [% END -%]
+    </pre>
+    </p>
+[% ELSE -%]
+  [% c.loc('No branding specified, using standard branding.') %]
+[% END -%]
 
 [% IF branding_edit_flag == 1 -%]
 [%