From 650b0faad71aaef786962dd4c9f0eb4ea4a8594d Mon Sep 17 00:00:00 2001
From: jakubnakielskixebia <jakub.nakielski@xebia.com>
Date: Tue, 28 Nov 2023 11:33:11 +0100
Subject: [PATCH 1/5] imp: meeting hide

---
 src/xebia_email_signature/templates/new_form.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/xebia_email_signature/templates/new_form.html b/src/xebia_email_signature/templates/new_form.html
index 27520fc..50c2d90 100644
--- a/src/xebia_email_signature/templates/new_form.html
+++ b/src/xebia_email_signature/templates/new_form.html
@@ -198,12 +198,12 @@ <h1 class="site-title">Email Signature Generator</h1>
                         >
                           My articles in the blog
                         </option>
-                        <option
+                        <!-- <option
                           data-custom-properties="/static/cta-icon-meeting.png"
                           value="meeting"
                         >
                           Schedule a meeting
-                        </option>
+                        </option> -->
                         <option
                           data-custom-properties="/static/cta-icon-event.png"
                           value="event"
@@ -261,7 +261,7 @@ <h1 class="site-title">Email Signature Generator</h1>
                 data-clone-max-items="2"
               >
                 <img src="static/icon-btn-add.svg" alt="Icon add" />
-                Add another Call to Action
+                Add Call to Action
               </button>
             </div>
           </div>

From 4b00d79fc6e63cc8185a801d045c5f566edd5aa9 Mon Sep 17 00:00:00 2001
From: jakubnakielskixebia <jakub.nakielski@xebia.com>
Date: Tue, 28 Nov 2023 11:55:05 +0100
Subject: [PATCH 2/5] imp: empty social media filter at controller layer

---
 src/xebia_email_signature/new_signature.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/xebia_email_signature/new_signature.py b/src/xebia_email_signature/new_signature.py
index bcc0a45..29aaf9c 100755
--- a/src/xebia_email_signature/new_signature.py
+++ b/src/xebia_email_signature/new_signature.py
@@ -199,6 +199,10 @@ def add_social_media(contact_details: dict) -> dict:
             if v and v[:8] != 'https://':
                 v = 'https://' + v
         formatted_social_media[int(split_key[1][:-1])][split_key[2][:-1]] = v
+    formatted_social_media = [
+        sm for sm in formatted_social_media 
+        if any(sm.values()) and sm.get('icon') and sm.get('link')
+    ]
     result["social_media"] = formatted_social_media
     return result
 

From b1fe9d6b08c0007c75a49889ad470caa393ff960 Mon Sep 17 00:00:00 2001
From: jakubnakielskixebia <jakub.nakielski@xebia.com>
Date: Tue, 28 Nov 2023 13:57:19 +0100
Subject: [PATCH 3/5] imp: form group infos

---
 .../templates/new_form.html                   | 31 ++++++++++---------
 1 file changed, 16 insertions(+), 15 deletions(-)

diff --git a/src/xebia_email_signature/templates/new_form.html b/src/xebia_email_signature/templates/new_form.html
index 50c2d90..3207fdc 100644
--- a/src/xebia_email_signature/templates/new_form.html
+++ b/src/xebia_email_signature/templates/new_form.html
@@ -69,13 +69,13 @@ <h1 class="site-title">Email Signature Generator</h1>
           </div>
 
           <div class="form-group">
-            <label for="unit"
-              >Unit
-              <span
-                >Optional: If you leave this blank, this field is not going to
-                appear</span
-              ></label
-            >
+            <label for="unit">
+              Unit
+              <span>
+                Optional: If you leave this blank, this field is not going to
+                appear
+              </span>
+            </label>
             <input
               type="text"
               id="unit"
@@ -86,7 +86,13 @@ <h1 class="site-title">Email Signature Generator</h1>
           </div>
 
           <div class="form-group">
-            <label for="phone">Phone</label>
+            <label for="phone">
+              Phone
+              <span>
+                Optional: If you leave this blank, this field is not going to
+                appear
+              </span>
+            </label>
             <input
               type="text"
               id="phone"
@@ -127,7 +133,7 @@ <h1 class="site-title">Email Signature Generator</h1>
                       type="text"
                       id="sm-0-link"
                       name="sm[0][link]"
-                      placeholder="https://www.linkedin.com/company/profile"
+                      placeholder="https://www.linkedin.com/in/username/"
                     />
                     <div id="sm-link-error" class="error"></div>
                   </div>
@@ -162,12 +168,7 @@ <h1 class="site-title">Email Signature Generator</h1>
 
           <div>
             <div>
-              <label for="cta-0-desc">
-                Call to action
-                <span
-                  >By default is generated this placeholder information</span
-                >
-              </label>
+              <label for="cta-0-desc">Call to action</label>
               <div class="js-form-cta" data-clone-count="0">
                 <div class="form-wrapper flex flex-column mb-4">
                   <div class="form-section">

From e657461030e4cf72381f6507320c341fea3e3b29 Mon Sep 17 00:00:00 2001
From: jakubnakielskixebia <jakub.nakielski@xebia.com>
Date: Tue, 28 Nov 2023 15:51:13 +0100
Subject: [PATCH 4/5] feat: logo as link to website

---
 .../templates/new_signature.html.jinja                    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/xebia_email_signature/templates/new_signature.html.jinja b/src/xebia_email_signature/templates/new_signature.html.jinja
index 42090f2..d6e75fe 100644
--- a/src/xebia_email_signature/templates/new_signature.html.jinja
+++ b/src/xebia_email_signature/templates/new_signature.html.jinja
@@ -11,8 +11,12 @@
   <table summary="contact details">
     <tr style="line-height: 1">
       <td style="padding-right: 10px">
-        <img width="75" height="70" alt="Xebia logo bulb"
-          src="{{ url_for('static', _external=True, _scheme=data['scheme'], filename='xebia-bulb.png') }}" />
+        <a href="https://xebia.com/" title="Website" style="text-decoration: none;">
+          <span style="text-decoration: none;">
+            <img width="75" height="70" alt="Xebia logo bulb"
+              src="{{ url_for('static', _external=True, _scheme=data['scheme'], filename='xebia-bulb.png') }}" />
+          </span>
+        </a>
       </td>
       <td colspan="2" style="
               vertical-align: top;

From 9f93d515eb76c797f936cd7dce920ff6b8c92657 Mon Sep 17 00:00:00 2001
From: jakubnakielskixebia <jakub.nakielski@xebia.com>
Date: Tue, 28 Nov 2023 16:45:26 +0100
Subject: [PATCH 5/5] feat: placeholder change on icon select

---
 .../templates/new_form.html                   | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/src/xebia_email_signature/templates/new_form.html b/src/xebia_email_signature/templates/new_form.html
index 3207fdc..440ec67 100644
--- a/src/xebia_email_signature/templates/new_form.html
+++ b/src/xebia_email_signature/templates/new_form.html
@@ -400,6 +400,7 @@ <h2 class="preview-header">Preview:</h2>
           cloneRemoveBtnInit(removeBtn);
         }
 
+        onSmSelectChangeInit();
         allMaxCharsCounterInit();
         allSelectInit();
       }
@@ -656,12 +657,33 @@ <h2 class="preview-header">Preview:</h2>
         phoneEl.value = phoneVal.replace(/[^\d\+]/g, '');
       }
 
+      function onSmSelectChangeInit() {
+        let smSelects = document.querySelectorAll('select[name^=sm]');
+        smSelects.forEach((smSelect) => {
+          smSelect.addEventListener('change', (e) =>
+            handleSmSelectChange(smSelect)
+          );
+        });
+      }
+
+      function handleSmSelectChange(smSelect) {
+        let linkFieldName = smSelect.name.replace('icon', 'link');
+        let linkField = document.querySelector(`[name="${linkFieldName}"]`);
+
+        if (smSelect.value === 'linkedin') {
+          linkField.placeholder = 'https://www.linkedin.com/in/username/';
+        } else if (smSelect.value === 'x') {
+          linkField.placeholder = 'https://x.com/profil';
+        }
+      }
+
       (() => {
         allCloneInit();
         allSelectInit();
         allMaxCharsCounterInit();
         signatureCopyInit();
         iframePrepareInit();
+        onSmSelectChangeInit();
       })();
     </script>
   </body>