From 2ae03c9852922ff23ce79910ce499bfb8c7644e6 Mon Sep 17 00:00:00 2001
From: Miquel Sabaté Solà <msabate@suse.com>
Date: Tue, 19 Mar 2024 11:19:53 +0100
Subject: thing: improved the UX on the form
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The "target" field is not autocompleted unless users explicitely
change its value themselves. The autocompletion is built from the
authors last name combo plus the year (which is rather naive, but
effective for most cases).

Moreover, some fields have been moved and the "access" field will only
appear if "url" is set.

Signed-off-by: Miquel Sabaté Solà <msabate@suse.com>
---
 app/javascript/controllers/thing_controller.js | 63 ++++++++++++++++++++++++++
 1 file changed, 63 insertions(+)

(limited to 'app/javascript/controllers')

diff --git a/app/javascript/controllers/thing_controller.js b/app/javascript/controllers/thing_controller.js
index 1518216..9c19eb6 100644
--- a/app/javascript/controllers/thing_controller.js
+++ b/app/javascript/controllers/thing_controller.js
@@ -2,6 +2,67 @@ import { Controller } from "@hotwired/stimulus"
 
 // Gathers all the methods that are relevant when manipulating things.
 export default class extends Controller {
+  static targets = ["identifier"];
+
+  connect() {
+    // The "target" completion is achieved by managing updates from authors
+    // (first part) and year (last one). We have to account when connecting that
+    // we might come from a reload of the page, which would contain partial
+    // data. Hence, this ought to be initialized with the current values from
+    // their respective inputs.
+    this.firstPart = document.getElementById("thing_authors").value.trim();
+    this.lastPart = document.getElementById("thing_year").value.trim();
+
+    // Whether autocompletion has been canceled or not. This will be forced to
+    // true when `#cancel` is called (i.e. the user has manually entered a value
+    // for "target"), or if we are coming from a page that already had data
+    // filled in.
+    this.canceled = this.firstPart !== '' && this.lastPart !== '' &&
+      document.getElementById("thing_target").value.trim() !== '';
+  }
+
+  // Update the information we get from the authors field so to autocomplete it
+  // into the "target" field if possible.
+  authorsUpdate() {
+    if (this.canceled) {
+      return;
+    }
+
+    const text = document.getElementById("thing_authors").value.trim();
+    if (text === '') {
+      return;
+    }
+
+    const authors = text.split(',').map(function(author) {
+      return author.split(' ').pop().trim();
+    });
+
+    this.firstPart = authors.join('');
+    this.identifierTarget.value = `${this.firstPart}${this.lastPart}`;
+  }
+
+  // Update the information we get from the year field so to autocomplete it
+  // into the "target" field if possible.
+  yearUpdate() {
+    if (this.canceled) {
+      return;
+    }
+
+    const text = document.getElementById("thing_year").value.trim();
+    if (text === '') {
+      return;
+    }
+
+    this.lastPart = text;
+    this.identifierTarget.value = `${this.firstPart}${this.lastPart}`;
+  }
+
+  // The user is writing into the "target" field directly: cancel autocompletion
+  // altogether.
+  cancel() {
+    this.canceled = true;
+  }
+
   // Update the external link depending on the changed value. The link will also
   // be hidden/shown depending if the current URL is empty or not.
   updateLink() {
@@ -9,8 +70,10 @@ export default class extends Controller {
 
     if (cur === "") {
       document.getElementById("thing-external-url-id").classList.add('hidden');
+      document.getElementById("thing-access-div").classList.add('hidden');
     } else {
       document.getElementById("thing-external-url-id").classList.remove('hidden');
+      document.getElementById("thing-access-div").classList.remove('hidden');
       document.getElementById("thing-external-url-id").href = cur;
     }
   }
-- 
cgit v1.2.3