From 509141ebe3e3151a6b813607142c18f2be80cd3e Mon Sep 17 00:00:00 2001
From: chme <christian.meffert@googlemail.com>
Date: Tue, 28 May 2019 22:48:58 +0200
Subject: [PATCH] [web-src] change styling of url-modal, default to play the
 stream and set focus after showing the modal

---
 .../components/ModalDialogAddUrlStream.vue    | 87 ++++++++++++++-----
 web-src/src/pages/PageQueue.vue               |  6 +-
 2 files changed, 69 insertions(+), 24 deletions(-)

diff --git a/web-src/src/components/ModalDialogAddUrlStream.vue b/web-src/src/components/ModalDialogAddUrlStream.vue
index d39fe68c..43b3ce98 100644
--- a/web-src/src/components/ModalDialogAddUrlStream.vue
+++ b/web-src/src/components/ModalDialogAddUrlStream.vue
@@ -3,23 +3,40 @@
     <transition name="fade">
       <div class="modal is-active" v-if="show">
         <div class="modal-background" @click="$emit('close')"></div>
-        <div class="modal-card">
-          <section class="modal-card-body">
-            <form v-on:submit.prevent="add_stream">
-              <div class="field">
-                <p class="control is-expanded has-icons-left">
-                  <input class="input is-rounded is-shadowless" type="text" placeholder="URL stream" v-model="url" ref="url_field">
-                  <span class="icon is-left">
-                    <i class="mdi mdi-file-music"></i>
-                  </span>
-                </p>
-              </div>
-            </form>
-          </section>
-          <footer class="modal-card-foot">
-            <button class="button is-success" @click="add_stream" :disabled="url.length < 9">Add</button>
-            <button class="button" @click="$emit('close')">Cancel</button>
-          </footer>
+        <div class="modal-content fd-modal-card">
+          <div class="card">
+            <div class="card-content">
+              <p class="title is-4">
+                Add stream URL
+              </p>
+              <form v-on:submit.prevent="play" class="fd-has-margin-bottom">
+                <div class="field">
+                  <p class="control is-expanded has-icons-left">
+                    <input class="input is-shadowless" type="text" placeholder="http://url-to-stream" v-model="url" :disabled="loading" ref="url_field">
+                    <span class="icon is-left">
+                      <i class="mdi mdi-web"></i>
+                    </span>
+                  </p>
+                </div>
+              </form>
+            </div>
+            <footer class="card-footer" v-if="loading">
+              <a class="card-footer-item has-text-dark">
+                <span class="icon"><i class="mdi mdi-web"></i></span> <span class="is-size-7">Loading ...</span>
+              </a>
+            </footer>
+            <footer class="card-footer" v-else>
+              <a class="card-footer-item has-text-danger" @click="$emit('close')">
+                <span class="icon"><i class="mdi mdi-cancel"></i></span> <span class="is-size-7">Cancel</span>
+              </a>
+              <a class="card-footer-item has-text-dark" @click="add_stream">
+                <span class="icon"><i class="mdi mdi-playlist-plus"></i></span> <span class="is-size-7">Add</span>
+              </a>
+              <a class="card-footer-item has-background-info has-text-white has-text-weight-bold" @click="play">
+                <span class="icon"><i class="mdi mdi-play"></i></span> <span class="is-size-7">Play</span>
+              </a>
+            </footer>
+          </div>
         </div>
         <button class="modal-close is-large" aria-label="close" @click="$emit('close')"></button>
       </div>
@@ -36,15 +53,43 @@ export default {
 
   data () {
     return {
-      url: ''
+      url: '',
+      loading: false
     }
   },
 
   methods: {
     add_stream: function () {
-      this.$emit('close')
-      webapi.queue_add(this.url)
-      this.url = ''
+      this.loading = true
+      webapi.queue_add(this.url).then(() => {
+        this.$emit('close')
+        this.url = ''
+      }).catch(() => {
+        this.loading = false
+      })
+    },
+
+    play: function () {
+      this.loading = true
+      webapi.player_play_uri(this.url, false).then(() => {
+        this.$emit('close')
+        this.url = ''
+      }).catch(() => {
+        this.loading = false
+      })
+    }
+  },
+
+  watch: {
+    'show' () {
+      if (this.show) {
+        this.loading = false
+
+        // We need to delay setting the focus to the input field until the field is part of the dom and visible
+        setTimeout(() => {
+          this.$refs.url_field.focus()
+        }, 10)
+      }
     }
   }
 }
diff --git a/web-src/src/pages/PageQueue.vue b/web-src/src/pages/PageQueue.vue
index 8fdd1d2a..a468b3d5 100644
--- a/web-src/src/pages/PageQueue.vue
+++ b/web-src/src/pages/PageQueue.vue
@@ -12,11 +12,11 @@
           </span>
           <span>Hide previous</span>
         </a>
-        <a class="button is-small" @click="add_stream_dialog">
+        <a class="button is-small" @click="open_add_stream_dialog">
           <span class="icon">
             <i class="mdi mdi-web"></i>
           </span>
-          <span>Add URL Stream</span>
+          <span>Add Stream</span>
         </a>
         <!--
         <a class="button" :class="{ 'is-info': edit_mode }" @click="edit_mode = !edit_mode">
@@ -133,7 +133,7 @@ export default {
       this.show_details_modal = true
     },
 
-    add_stream_dialog: function (item) {
+    open_add_stream_dialog: function (item) {
       this.show_url_modal = true
     }
   }