From 4ddbcf4037a10ab323e8bc5477f4a518b25a4733 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien.fastre@champs-libres.coop>
Date: Wed, 19 Nov 2025 16:45:15 +0100
Subject: [PATCH] Refactor SCSS and Vue components to standardize "required"
 labels.

- Moved SCSS rules for `.required` labels to a shared scope in `forms.scss`.
- Updated labels in `PersonEdit.vue` to consistently use the `required` class.
---
 .../Resources/public/chill/scss/forms.scss         | 14 +++++++++-----
 .../vuejs/_components/OnTheFly/PersonEdit.vue      | 10 +++++-----
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/src/Bundle/ChillMainBundle/Resources/public/chill/scss/forms.scss b/src/Bundle/ChillMainBundle/Resources/public/chill/scss/forms.scss
index 28c597bc0..daa22170f 100644
--- a/src/Bundle/ChillMainBundle/Resources/public/chill/scss/forms.scss
+++ b/src/Bundle/ChillMainBundle/Resources/public/chill/scss/forms.scss
@@ -29,10 +29,14 @@ form {
 
     label {
         display: inline;
-        &.required:after {
-            content: " *";
-            color: $red;
-        }
+    }
+}
+
+label {
+    display: inline;
+    &.required:after {
+        content: " *";
+        color: $red;
     }
 }
 
@@ -45,4 +49,4 @@ form {
 
 .chill_filter_order {
     background: $gray-100;
-}
\ No newline at end of file
+}
diff --git a/src/Bundle/ChillPersonBundle/Resources/public/vuejs/_components/OnTheFly/PersonEdit.vue b/src/Bundle/ChillPersonBundle/Resources/public/vuejs/_components/OnTheFly/PersonEdit.vue
index 077ded389..f131b4c6f 100644
--- a/src/Bundle/ChillPersonBundle/Resources/public/vuejs/_components/OnTheFly/PersonEdit.vue
+++ b/src/Bundle/ChillPersonBundle/Resources/public/vuejs/_components/OnTheFly/PersonEdit.vue
@@ -10,7 +10,7 @@
             v-model="lastName"
             :placeholder="trans(PERSON_MESSAGES_PERSON_LASTNAME)"
           />
-          <label for="lastname">{{
+          <label for="lastname" class="required">{{
             trans(PERSON_MESSAGES_PERSON_LASTNAME)
           }}</label>
         </div>
@@ -45,7 +45,7 @@
             v-model="firstName"
             :placeholder="trans(PERSON_MESSAGES_PERSON_FIRSTNAME)"
           />
-          <label for="firstname">{{
+          <label for="firstname" class="required">{{
             trans(PERSON_MESSAGES_PERSON_FIRSTNAME)
           }}</label>
         </div>
@@ -104,7 +104,7 @@
               :placeholder="localizeString(worker.label)"
               @input="onIdentifierInput($event, worker.definition_id)"
             />
-            <label :for="'worker_' + worker.definition_id">{{
+            <label :for="'worker_' + worker.definition_id" :class="{required: worker.presence == 'REQUIRED'}">{{
                 localizeString(worker.label)
               }}</label>
           </div>
@@ -134,7 +134,7 @@
               {{ g.label }}
             </option>
           </select>
-          <label for="gender" class="form-label">{{
+          <label for="gender" class="required">{{
             trans(PERSON_MESSAGES_PERSON_GENDER_TITLE)
           }}</label>
         </div>
@@ -163,7 +163,7 @@
               {{ c.name }}
             </option>
           </select>
-          <label for="center">{{ trans(PERSON_MESSAGES_PERSON_CENTER_TITLE) }}</label>
+          <label for="center" class="required">{{ trans(PERSON_MESSAGES_PERSON_CENTER_TITLE) }}</label>
         </div>
         <div
           v-for="err in violations.violationTitles('center')"