From bde87c7afb50dd5f736b828cd5388811d203a32d Mon Sep 17 00:00:00 2001
From: ryan <ryanrain@gmail.com>
Date: Sat, 15 Dec 2018 05:09:19 -0600
Subject: [PATCH] podcast smartblock generation: add success message and
 conditional hide

---
 .../views/scripts/podcast/podcast.phtml       | 40 ++++----
 airtime_mvc/public/css/styles.css             | 95 +++++++------------
 .../public/js/airtime/library/podcast.js      | 11 ++-
 3 files changed, 65 insertions(+), 81 deletions(-)

diff --git a/airtime_mvc/application/views/scripts/podcast/podcast.phtml b/airtime_mvc/application/views/scripts/podcast/podcast.phtml
index 6c715d8b3..b13d88f3f 100644
--- a/airtime_mvc/application/views/scripts/podcast/podcast.phtml
+++ b/airtime_mvc/application/views/scripts/podcast/podcast.phtml
@@ -7,28 +7,34 @@
     <div class="inner_editor_wrapper">
         <form class="podcast-metadata">
             <input ng-value="podcast.id" class="obj_id" type="hidden"/>
-            <div>
+            <div class="podcast-metadata-row">
                 <label for="podcast_name"><?php echo _("Podcast Name: ") ?></label>
-                <input name="podcast_name" type="text" ng-model="podcast.title" value={{podcast.title}} class="podcast-metadata-field">
+                <input name="podcast_name" type="text" ng-model="podcast.title" value={{podcast.title}} >
             </div>
-            <div>
+            <div class="podcast-metadata-row">
                 <label for="podcast_url"><?php echo _("Podcast URL: ") ?></label>
-                <a href="{{podcast.url}}" target="_blank">
-                    <span class="podcast-metadata-field">{{podcast.url}}</span>
-                </a>
+                <a href="{{podcast.url}}" target="_blank" class="podcast-url">{{podcast.url}}</a>
             </div>
-            <div style="padding-top: 0.1em;">
-                <label for="podcast_auto_ingest"><input name="podcast_auto_ingest" ng-model="podcast.auto_ingest" type="checkbox" class="float-right"/></label>
-                <span class="podcast-metadata-field"><?php echo _("Automatically download latest episodes?") ?></span>
+            <div class="podcast-metadata-row">
+                <div>
+                    <input name="podcast_auto_ingest"  id="podcast_auto_ingest" ng-model="podcast.auto_ingest" type="checkbox"/>
+                </div>
+                <label for="podcast_auto_ingest"><?php echo _("Automatically download latest episodes") ?></label>
             </div>
-            <div style="padding-top: 0.1em;">
-                <label for="podcast_album_override"><input name="podcast_album_override" ng-model="podcast.album_override" type="checkbox" class="no-float"/></label>
-                <span class="podcast-metadata-field"><?php echo _("Override album name with podcast name during ingest."); ?></span>
+            <div class="podcast-metadata-row">
+                <div>
+                    <input name="podcast_album_override" id="podcast_album_override" ng-model="podcast.album_override" type="checkbox" />
+                </div>
+                <label for="podcast_album_override"><?php echo _("Overwrite album name"); ?></label>
             </div>
-            <div style="padding-top: 0.1em;">
-                <button ng-click="createSmartblock()" class="btn" type="button" name="smartblock">
-                    <?php echo _("Generate Smartblock and Playlist for Podcast") ?>
-                </button>
+            <div class="podcast-metadata-row" >
+                <div></div>
+                <div>
+                    <button ng-click="createSmartblock()" ng-disabled="!podcast.album_override" class="btn" type="button" name="smartblock">
+                        <?php echo _("Generate Smartblock and Playlist for Podcast") ?>
+                    </button>
+                    <div class='pc-sb-success success' style='display:none'></span></div>
+                </div>
             </div>
         </form>
     </div>
@@ -46,6 +52,6 @@
                 <?php echo _("Save") ?>
             </button>
         </div>
-        <div class='success' style='display:none'></span></div>
+        <div class='success' style='display:none'></div>
     </div>
 </div>
diff --git a/airtime_mvc/public/css/styles.css b/airtime_mvc/public/css/styles.css
index 3cab8afd2..2658c7c23 100644
--- a/airtime_mvc/public/css/styles.css
+++ b/airtime_mvc/public/css/styles.css
@@ -4137,66 +4137,34 @@ li .ui-state-hover {
     padding: 5px 0 0 5px;
 }
 
-.podcast-metadata p {
-    text-align: center;
-    padding: 0 10px;
-    margin-top: 0;
+.podcast-metadata-row {
+    margin: 10px 0;
+    display: flex;
+    align-items: center;
 }
-
-.podcast-metadata label,
-.podcast-metadata input:not([type="checkbox"]),
-.podcast-metadata select,
-.podcast-metadata textarea,
-.media-metadata label,
-.media-metadata input:not([type="checkbox"]),
-.media-metadata select,
-.media-metadata textarea {
-    margin: 4px 0;
-}
-
-.podcast-metadata input[type="checkbox"] {
-    float: left;
-    margin: 8px 0 0;
-}
-
-.podcast-metadata input[type="checkbox"].no-float {
+.podcast-metadata .podcast-metadata-row * {
+    line-height: 1.3;
+    position: static;
     float: none;
-    margin: 8px 0 0;
+}
+.podcast-metadata-row > *:first-child {
+    flex: 0 0 25%;
+    max-width: 25%;
+    text-align: right;
+    padding-right: 2%;
+    box-sizing: border-box;
+}
+.podcast-metadata-row > *:last-child {
+    flex: 0 0 75%;
+    max-width: 75%;
+    box-sizing: border-box;
+    text-align: left;
 }
 
-.podcast-metadata input[type="checkbox"].float-right {
-    float:right;
-    margin: 8px 0 0;
-
-}
-
-.podcast-metadata label,
-.media-metadata label {
-    display: block;
-    width: 20%;
-    float: left;
-}
-
-.podcast-metadata input:not([type="checkbox"]),
-.podcast-metadata select,
-.podcast-metadata textarea,
-.media-metadata input:not([type="checkbox"]),
-.media-metadata select,
-.media-metadata textarea {
-    width: 60%;
-    float: left;
-}
-
-.podcast-metadata-field {
-    display: inline-block;
-    width: 60%;
-    min-width: 60%;
-    max-width: 60%;
+.podcast-url {
     white-space: nowrap;
     overflow: hidden;
     text-overflow: ellipsis;
-    margin: 4px 0;
-    line-height: 24px;
 }
 
 .podcast_episodes_imported {
@@ -4221,6 +4189,17 @@ li .ui-state-hover {
     overflow: auto;
 }
 
+
+#podcast_url_dialog table tr td {
+    padding: 0;
+}
+
+#podcast_url_dialog .errors {
+    display: inline-block;
+    float: right;
+    width: 50%;
+}
+
 /* UI Revamp Video */
 
 #whatsnew {
@@ -4336,16 +4315,6 @@ li .ui-state-hover {
     float: right;
 }
 
-#podcast_url_dialog table tr td {
-    padding: 0;
-}
-
-#podcast_url_dialog .errors {
-    display: inline-block;
-    float: right;
-    width: 50%;
-}
-
 #upgrade-feature-locked {
     margin: 0 auto;
     width: 50%;
diff --git a/airtime_mvc/public/js/airtime/library/podcast.js b/airtime_mvc/public/js/airtime/library/podcast.js
index b4aad6219..52e35736d 100644
--- a/airtime_mvc/public/js/airtime/library/podcast.js
+++ b/airtime_mvc/public/js/airtime/library/podcast.js
@@ -61,7 +61,16 @@ var AIRTIME = (function (AIRTIME) {
                     id: $scope.podcast.id,
                     title: $scope.podcast.title
                 }, 
-                callback
+                function() {
+                    var successMsg = $('.active-tab .pc-sb-success')
+                    successMsg.text($.i18n._('Smartblock and playlist generated'));
+                    successMsg.show();
+                    setTimeout(function(){
+                        successMsg.hide();
+                    }, 5000);
+                    dt = $('table[id="library_display"]').dataTable();
+                    dt.fnStandingRedraw();
+                }
             );
             // save podcast
             $scope.savePodcast();