From 006c771d4a085d6b51a55e5ad48abbd064fc8536 Mon Sep 17 00:00:00 2001
From: Guillaume Grossetie <ggrossetie@yuzutech.fr>
Date: Fri, 29 Nov 2024 16:19:23 +0100
Subject: [PATCH] =?UTF-8?q?chore:=20marge=20suppl=C3=A9mentaire=20sur=20le?=
 =?UTF-8?q?=20formulaire=20des=20personnes?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 front/src/components/Form.jsx            |  2 -
 front/src/components/form.module.scss    | 15 +++---
 front/src/schemas/article-ui-schema.json | 62 ++++++++----------------
 3 files changed, 27 insertions(+), 52 deletions(-)

diff --git a/front/src/components/Form.jsx b/front/src/components/Form.jsx
index f8108eb45..7f70937fd 100644
--- a/front/src/components/Form.jsx
+++ b/front/src/components/Form.jsx
@@ -97,7 +97,6 @@ function ArrayFieldTemplate (properties) {
   const addItemTitle = properties.uiSchema['ui:add-item-title'] ?? 'form.addItem.title'
   const removeItemTitle = properties.uiSchema['ui:remove-item-title'] ?? 'form.removeItem.title'
   const title = properties.uiSchema['ui:title']
-
   const inlineRemoveButton = properties.schema?.items?.type === 'string' || !removeItemTitle
   return (
       <fieldset className={clsx(styles.fieldset, styles.array)} key={properties.key}>
@@ -153,7 +152,6 @@ function FieldTemplate (properties) {
   const label = properties.schema.$id
     ? properties.label[properties.schema.$id]
     : properties.label
-  console.log('FieldTemplate', {id, properties, children})
   return (
     <div className={classNames} style={style}>
       {displayLabel && <label htmlFor={id}>
diff --git a/front/src/components/form.module.scss b/front/src/components/form.module.scss
index 5ee52baf7..a417539aa 100644
--- a/front/src/components/form.module.scss
+++ b/front/src/components/form.module.scss
@@ -74,7 +74,6 @@
         > fieldset {
           > button {
             font-weight: bold;
-            margin: 0.5rem 0;
           }
         }
       }
@@ -108,12 +107,16 @@
         > * {
           width: 100%;
         }
+
+        &.person {
+          margin-top: 1rem;
+        }
       }
 
       &.field-string,
       &.field-boolean {
         label {
-         padding-bottom: 0.15rem;
+          padding-bottom: 0.15rem;
         }
       }
 
@@ -160,11 +163,11 @@
           }
         }
       }
-      
+
       .array-item.keywords {
         display: flex;
         align-items: center;
-        
+
         > .field-string {
           flex: 1;
         }
@@ -230,10 +233,6 @@
   }
 }
 
-.array {
-  gap: 1.5rem;
-}
-
 .comboboxReadonlyField {
   margin-right: .5em;
 }
diff --git a/front/src/schemas/article-ui-schema.json b/front/src/schemas/article-ui-schema.json
index 38b15893b..c5527b423 100644
--- a/front/src/schemas/article-ui-schema.json
+++ b/front/src/schemas/article-ui-schema.json
@@ -115,6 +115,7 @@
   "authors": {
     "ui:add-item-title": "form.addPerson.title",
     "items": {
+      "ui:classNames": "person",
       "ui:autocomplete": "IsidoreAuthorSearch",
       "surname": {
         "ui:title": "article.metadata.people.surname"
@@ -126,7 +127,8 @@
         "ui:title": "article.metadata.people.affiliations"
       },
       "biography": {
-        "ui:title": "article.metadata.people.biography"
+        "ui:title": "article.metadata.people.biography",
+        "ui:widget": "textarea"
       },
       "email": {
         "ui:title": "article.metadata.people.email"
@@ -226,6 +228,7 @@
   "keywords": {
     "ui:add-item-title": "article.metadata.keywords.addItem",
     "ui:remove-item-title": "article.metadata.keywords.removeItem",
+    "ui:className": "keywords",
     "items": {
       "ui:label": false,
       "ui:className": "keywords"
@@ -270,6 +273,7 @@
   "issueDirectors": {
     "ui:add-item-title": "form.addPerson.title",
     "items": {
+      "ui:classNames": "person",
       "surname": {
         "ui:title": "article.metadata.people.surname"
       },
@@ -280,42 +284,8 @@
         "ui:title": "article.metadata.people.affiliations"
       },
       "biography": {
-        "ui:title": "article.metadata.people.biography"
-      },
-      "email": {
-        "ui:title": "article.metadata.people.email"
-      },
-      "orcid": {
-        "ui:title": "article.metadata.people.orcid"
-      },
-      "viaf": {
-        "ui:title": "article.metadata.people.viaf"
-      },
-      "foaf": {
-        "ui:title": "article.metadata.people.foaf"
-      },
-      "isni": {
-        "ui:title": "article.metadata.people.isni"
-      },
-      "wikidata": {
-        "ui:title": "article.metadata.people.wikidata"
-      }
-    }
-  },
-  "director": {
-    "ui:add-item-title": "form.addPerson.title",
-    "items": {
-      "surname": {
-        "ui:title": "article.metadata.people.surname"
-      },
-      "forename": {
-        "ui:title": "article.metadata.people.forename"
-      },
-      "affiliations": {
-        "ui:title": "article.metadata.people.affiliations"
-      },
-      "biography": {
-        "ui:title": "article.metadata.people.biography"
+        "ui:title": "article.metadata.people.biography",
+        "ui:widget": "textarea"
       },
       "email": {
         "ui:title": "article.metadata.people.email"
@@ -340,6 +310,7 @@
   "transcribers": {
     "ui:add-item-title": "form.addPerson.title",
     "items": {
+      "ui:classNames": "person",
       "surname": {
         "ui:title": "article.metadata.people.surname"
       },
@@ -350,7 +321,8 @@
         "ui:title": "article.metadata.people.affiliations"
       },
       "biography": {
-        "ui:title": "article.metadata.people.biography"
+        "ui:title": "article.metadata.people.biography",
+        "ui:widget": "textarea"
       },
       "email": {
         "ui:title": "article.metadata.people.email"
@@ -387,6 +359,7 @@
   "translators": {
     "ui:add-item-title": "form.addPerson.title",
     "items": {
+      "ui:classNames": "person",
       "surname": {
         "ui:title": "article.metadata.people.surname"
       },
@@ -397,7 +370,8 @@
         "ui:title": "article.metadata.people.affiliations"
       },
       "biography": {
-        "ui:title": "article.metadata.people.biography"
+        "ui:title": "article.metadata.people.biography",
+        "ui:widget": "textarea"
       },
       "email": {
         "ui:title": "article.metadata.people.email"
@@ -426,6 +400,7 @@
   "reviewers": {
     "ui:add-item-title": "form.addPerson.title",
     "items": {
+      "ui:classNames": "person",
       "surname": {
         "ui:title": "article.metadata.people.surname"
       },
@@ -436,7 +411,8 @@
         "ui:title": "article.metadata.people.affiliations"
       },
       "biography": {
-        "ui:title": "article.metadata.people.biography"
+        "ui:title": "article.metadata.people.biography",
+        "ui:widget": "textarea"
       },
       "email": {
         "ui:title": "article.metadata.people.email"
@@ -484,6 +460,7 @@
         "ui:widget": "textarea"
       },
       "keywords": {
+        "ui:className": "keywords",
         "ui:title": "article.metadata.keywords",
         "ui:add-item-title": "article.metadata.keywords.addItem",
         "ui:remove-item-title": "article.metadata.keywords.removeItem",
@@ -506,7 +483,7 @@
   "journalDirectors": {
     "ui:add-item-title": "form.addPerson.title",
     "items": {
-      "ui:autocomplete": "IsidoreAuthorSearch",
+      "ui:classNames": "person",
       "surname": {
         "ui:title": "article.metadata.people.surname"
       },
@@ -517,7 +494,8 @@
         "ui:title": "article.metadata.people.affiliations"
       },
       "biography": {
-        "ui:title": "article.metadata.people.biography"
+        "ui:title": "article.metadata.people.biography",
+        "ui:widget": "textarea"
       },
       "email": {
         "ui:title": "article.metadata.people.email"