From 1f50229aa5a7e97a2df285540dddecbb7957df33 Mon Sep 17 00:00:00 2001
From: gskema <gytis@ymail.com>
Date: Mon, 2 May 2016 14:42:19 +0300
Subject: [PATCH 1/4] [*] JS: Clean up blocknewsletter JS

---
 .../blocknewsletter/blocknewsletter.js        | 23 ++++++++++++-------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js b/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js
index 8e6b864c..bbd67077 100644
--- a/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js
+++ b/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js
@@ -1,22 +1,29 @@
-$(document).ready(function() {
+$(function() {
 
   $('#newsletter-input').on({
     focus: function() {
-      if ($(this).val() == placeholder_blocknewsletter || $(this).val() == msg_newsl)
+      if ($(this).val() == placeholder_blocknewsletter || $(this).val() == msg_newsl) {
         $(this).val('');
+      }
     },
     blur: function() {
-      if ($(this).val() == '')
+      if ($(this).val() == '') {
         $(this).val(placeholder_blocknewsletter);
+      }
     }
   });
 
-  var cssClass = 'alert alert-danger';
-  if (typeof nw_error != 'undefined' && !nw_error)
-    cssClass = 'alert alert-success';
+  var alertClass = 'alert alert-danger';
+  if (typeof nw_error != 'undefined' && !nw_error) {
+    alertClass = 'alert alert-success';
+  }
 
   if (typeof msg_newsl != 'undefined' && msg_newsl) {
-    $('#columns').prepend('<div class="clearfix"></div><p class="' + cssClass + '"> ' + alert_blocknewsletter + '</p>');
-    $('html, body').animate({scrollTop: $('#columns').offset().top}, 'slow');
+    var $cols =  $('#columns');
+    $cols.prepend('<div class="' + alertClass + '">' + alert_blocknewsletter + '</div>');
+    $('html, body').animate({
+      scrollTop: $cols.offset().top
+    }, 'slow');
   }
+
 });

From ae21596f9d1f88df1880d37fc42f9de5d0bc13a5 Mon Sep 17 00:00:00 2001
From: gskema <gytis@ymail.com>
Date: Mon, 2 May 2016 14:42:43 +0300
Subject: [PATCH 2/4] [*] FO: Make blocknewsletter input required and of email
 type

---
 .../modules/blocknewsletter/blocknewsletter.tpl                 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl b/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl
index 2bda160d..f846268d 100644
--- a/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl
+++ b/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl
@@ -3,7 +3,7 @@
   <form action="{$link->getPageLink('index', null, null, null, false, null, true)|escape:'html':'UTF-8'}" method="post">
     <div class="form-group{if isset($msg) && $msg } {if $nw_error}form-error{else}form-ok{/if}{/if}" >
       <div class="input-group">
-        <input class="form-control" id="newsletter-input" type="text" name="email" size="18" value="{if isset($msg) && $msg}{$msg}{elseif isset($value) && $value}{$value}{else}{l s='Enter your e-mail' mod='blocknewsletter'}{/if}" />
+        <input class="form-control" id="newsletter-input" type="email" name="email" size="18" value="{if isset($msg) && $msg}{$msg}{elseif isset($value) && $value}{$value}{else}{l s='Enter your e-mail' mod='blocknewsletter'}{/if}" required/>
         <span class="input-group-btn">
           <button type="submit" name="submitNewsletter" class="btn btn-primary">
             <i class="icon icon-chevron-right"></i>

From c56d1bef5b4fd54c3673e2a4789aa53b119bd3b1 Mon Sep 17 00:00:00 2001
From: gskema <gytis@ymail.com>
Date: Mon, 2 May 2016 14:47:44 +0300
Subject: [PATCH 3/4] [-] FO: Required attribute doesn't work with manual
 placeholder in blocknewsletter

---
 .../modules/blocknewsletter/blocknewsletter.tpl                 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl b/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl
index f846268d..7e2508d3 100644
--- a/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl
+++ b/themes/community-theme-16/modules/blocknewsletter/blocknewsletter.tpl
@@ -3,7 +3,7 @@
   <form action="{$link->getPageLink('index', null, null, null, false, null, true)|escape:'html':'UTF-8'}" method="post">
     <div class="form-group{if isset($msg) && $msg } {if $nw_error}form-error{else}form-ok{/if}{/if}" >
       <div class="input-group">
-        <input class="form-control" id="newsletter-input" type="email" name="email" size="18" value="{if isset($msg) && $msg}{$msg}{elseif isset($value) && $value}{$value}{else}{l s='Enter your e-mail' mod='blocknewsletter'}{/if}" required/>
+        <input class="form-control" id="newsletter-input" type="email" name="email" size="18" value="{if isset($msg) && $msg}{$msg}{elseif isset($value) && $value}{$value}{else}{l s='Enter your e-mail' mod='blocknewsletter'}{/if}"/>
         <span class="input-group-btn">
           <button type="submit" name="submitNewsletter" class="btn btn-primary">
             <i class="icon icon-chevron-right"></i>

From f90429800c1e013228c064052e7b978213d85dfe Mon Sep 17 00:00:00 2001
From: gskema <gytis@ymail.com>
Date: Mon, 2 May 2016 14:47:57 +0300
Subject: [PATCH 4/4] [+] FO: Add loading overlay to newsletter form

---
 .../js/modules/blocknewsletter/blocknewsletter.js             | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js b/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js
index bbd67077..3fd3d74e 100644
--- a/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js
+++ b/themes/community-theme-16/js/modules/blocknewsletter/blocknewsletter.js
@@ -1,5 +1,9 @@
 $(function() {
 
+  $('#blocknewsletter').find('form').on('submit', function () {
+    $(this).addClass('loading-overlay');
+  });
+
   $('#newsletter-input').on({
     focus: function() {
       if ($(this).val() == placeholder_blocknewsletter || $(this).val() == msg_newsl) {