From 8bf9df441906c9bf94e4d706fe138af1b64e3b4e Mon Sep 17 00:00:00 2001
From: glax <johanna@bernloehr.eu>
Date: Tue, 23 May 2023 18:46:06 +0200
Subject: [PATCH] Done Better Task-Adder

---
 Website/index.html     |  7 +++--
 Website/interaction.js | 69 ++++++++++++++++++++++++------------------
 Website/style.css      | 17 +++++++++--
 3 files changed, 59 insertions(+), 34 deletions(-)

diff --git a/Website/index.html b/Website/index.html
index ff20bb1..44b8c62 100644
--- a/Website/index.html
+++ b/Website/index.html
@@ -37,8 +37,8 @@
       </publication>
     </content>    
     
-    <popup>
-      <blur-background></blur-background>
+    <popup id="addTaskPopup">
+      <blur-background id="blurBackgroundTaskPopup"></blur-background>
       <addtask-window>
         <window-titlebar>
           <p>Add Task</p>
@@ -57,6 +57,9 @@
           <div id="taskSelectOutput"></div>
         </window-content>
       </addtask-window>
+    </popup>
+    <popup id="publicationViewerPopup">
+      <blur-background id="blurBackgroundPublicationPopup"></blur-background>
       <publication-viewer>
         <img id="pubviewcover" src="media/cover.jpg" alt="cover">
         <publication-information>
diff --git a/Website/interaction.js b/Website/interaction.js
index 1e21e3c..6df9fba 100644
--- a/Website/interaction.js
+++ b/Website/interaction.js
@@ -30,9 +30,9 @@ const settingsTab = document.querySelector("#settingstab");
 const settingsCog = document.querySelector("#settingscog");
 const selectRecurrence = document.querySelector("#selectReccurrence");
 const tasksContent = document.querySelector("content");
-const generalPopup = document.querySelector("popup");
-const addTaskWindow = document.querySelector("addtask-window");
-const publicationViewer = document.querySelector("publication-viewer");
+const addTaskPopup = document.querySelector("#addTaskPopup");
+const publicationViewerPopup = document.querySelector("#publicationViewerPopup");
+const publicationViewerWindow = document.querySelector("publication-viewer");
 const publicationViewerDescription = document.querySelector("#publicationViewerDescription");
 const publicationViewerName = document.querySelector("#publicationViewerName");
 const publicationViewerAuthor = document.querySelector("#publicationViewerAuthor");
@@ -42,10 +42,11 @@ const publicationAdd = document.querySelector("publication-add");
 const closetaskpopup = document.querySelector("#closePopupImg");
 
 settingsCog.addEventListener("click", () => slide());
-closetaskpopup.addEventListener("click", () => HidePopup());
-document.querySelector("blur-background").addEventListener("click", () => HidePopup());
+closetaskpopup.addEventListener("click", () => HideAddTaskPopup());
+document.querySelector("#blurBackgroundTaskPopup").addEventListener("click", () => HideAddTaskPopup());
+document.querySelector("#blurBackgroundPublicationPopup").addEventListener("click", () => HidePublicationPopup());
 publicationDelete.addEventListener("click", () => DeleteTaskClick());
-publicationAdd.addEventListener("click", () => CreateTask("DownloadNewChapters", selectRecurrence.value, connectorSelect.value, toEditId, "en"));
+publicationAdd.addEventListener("click", () => AddTaskClick());
 
 /*
 let availableTaskTypes;
@@ -74,18 +75,27 @@ GetAvailableControllers()
 
 searchPublicationQuery.addEventListener("keypress", (event) => {
    if(event.key === "Enter"){
+       selectRecurrence.disabled = true;
+       connectorSelect.disabled = true;
+       searchPublicationQuery.disabled = true;
+       
        selectPublication.replaceChildren();
        GetPublication(connectorSelect.value, searchPublicationQuery.value)
            //.then(json => console.log(json));
            .then(json => 
                json.forEach(publication => {
                    var option = CreatePublication(publication, connectorSelect.value);
-                   option.addEventListener("click", () => {
-                       ShowPublicationViewerWindow(publication.internalId, event, true);
+                   option.addEventListener("click", (mouseEvent) => {
+                       ShowPublicationViewerWindow(publication.internalId, mouseEvent, true);
                    });
                    selectPublication.appendChild(option);
                }
-           ));
+           ))
+           .then(() => {
+               selectRecurrence.disabled = false;
+               connectorSelect.disabled = false;
+               searchPublicationQuery.disabled = false;
+           });
    } 
 });
 
@@ -112,7 +122,12 @@ function CreatePublication(publication, connector){
 function DeleteTaskClick(){
     taskToDelete = tasks.filter(tTask => tTask.publication.internalId === toEditId)[0];
     DeleteTask("DownloadNewChapters", taskToDelete.connectorName, toEditId);
-    HidePopup();
+    HideAddTaskPopup();
+}
+
+function AddTaskClick(){
+    CreateTask("DownloadNewChapters", selectRecurrence.value, connectorSelect.value, toEditId, "en")
+    HideAddTaskPopup();
 }
 
 var slideIn = true;
@@ -134,15 +149,9 @@ function ResetContent(){
     add.addEventListener("click", () => ShowNewTaskWindow());
     tasksContent.appendChild(add);
 }
-
-function ShowPopup(){
-    generalPopup.style.display = "block";
-    generalPopup.animate(fadeIn, fadeInTiming);
-}
-
 function ShowPublicationViewerWindow(publicationId, event, add){
-    publicationViewer.style.top = `${event.clientY - 60}px`;
-    publicationViewer.style.left = `${event.clientX}px`;
+    publicationViewerWindow.style.top = `${event.clientY - 60}px`;
+    publicationViewerWindow.style.left = `${event.clientX}px`;
     var publication = publications.filter(pub => pub.internalId === publicationId)[0];
     
     publicationViewerName.innerText = publication.sortName;
@@ -152,28 +161,28 @@ function ShowPublicationViewerWindow(publicationId, event, add){
     toEditId = publicationId;
     
     if(add){
-        publicationAdd.style.display = "none";
-        publicationDelete.style.display = "block";
-    }
-    else{
         publicationAdd.style.display = "block";
         publicationDelete.style.display = "none";
     }
+    else{
+        publicationAdd.style.display = "none";
+        publicationDelete.style.display = "block";
+    }
     
     toEditId = publicationId;
-    publicationViewer.style.display = "block";
-    ShowPopup();
+    publicationViewerPopup.style.display = "block";
 }
 
 function ShowNewTaskWindow(){
     selectPublication.replaceChildren();
-    addTaskWindow.style.display = "flex";
-    ShowPopup();
+    addTaskPopup.style.display = "block";
 }
-function HidePopup(){
-    generalPopup.style.display = "none";
-    addTaskWindow.style.display = "none";
-    publicationViewer.style.display = "none";
+function HideAddTaskPopup(){
+    addTaskPopup.style.display = "none";
+}
+
+function HidePublicationPopup(){
+    publicationViewerPopup.style.display = "none";
 }
 
 const fadeIn = [
diff --git a/Website/style.css b/Website/style.css
index 1a848b4..e9220bf 100644
--- a/Website/style.css
+++ b/Website/style.css
@@ -226,7 +226,7 @@ blur-background {
 }
 
 addtask-window {
-    display: none;
+    display: flex;
     flex-direction: column;
     flex-wrap: nowrap;
     position: absolute;
@@ -307,8 +307,12 @@ addtask-settings addtask-setting{
     align-content: start;
 }
 
+#publicationViewerPopup{
+    z-index: 6;
+}
+
 publication-viewer{
-    display: none;
+    display: block;
     width: 500px;
     height: 300px;
     position: absolute;
@@ -375,4 +379,13 @@ publication-viewer publication-information publication-delete {
     color: red;
     margin: 20px;
     font-size: 16pt;
+}
+
+publication-viewer publication-information publication-add {
+    position: absolute;
+    bottom: 0px;
+    right: 0px;
+    color: limegreen;
+    margin: 20px;
+    font-size: 16pt;
 }
\ No newline at end of file