Merge pull request #319 from crf-devs/hide-included-user-skills

Remove included skills from users list

Author: mRoca

assets/css/planning.scss

@@ -23,10 +23,6 @@ $tableBoxSize: 40px;
   .planning {
     margin-bottom: 40px;
 
-    .badge {
-      margin-right: 2px;
-    }
-
     thead {
       th {
         font-weight: 600;

src/Form/Type/UserType.php

@@ -120,7 +120,7 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
             if (self::DISPLAY_ORGANIZATION === $options['display_type']) {
                 $form->add('vulnerable', ChoiceType::class, [
                     'choices' => [
-                        'fait PAS partie des personnes vulnérables' => 0,
+                        'ne fait PAS partie des personnes vulnérables' => 0,
                         'fait partie des personnes vulnérables' => 1,
                     ],
                     'expanded' => true,

src/Twig/Extension/UserExtension.php

@@ -31,6 +31,6 @@ public function formatBadge(string $skill): string
     {
         $importantSkills = $this->skillSetDomain->getImportantSkills();
 
-        return sprintf('<span class="badge badge-%s">%s</span>', \in_array($skill, $importantSkills, true) ? 'primary' : 'secondary', $skill);
+        return sprintf('<span class="badge badge-%s mr-1">%s</span>', \in_array($skill, $importantSkills, true) ? 'primary' : 'secondary', $skill);
     }
 }

templates/organization/user/_list.html.twig

@@ -16,15 +16,15 @@
         <tbody>
         {% for user in users %}
             <tr>
-                <td>{{ user.identificationNumber }}</td>
+                <td><small>{{ user.identificationNumber }}</small></td>
                 <td>{{ user.firstName }}</td>
                 <td>{{ user.lastName }}</td>
                 <td>
-                    {% for skill in user.skillSet -%}
+                    {% for skill in getDisplayableSkillsInPlanning(user) -%}
                         {{ skill|skillBadge }}
                     {%- endfor %}
                 </td>
-                <td>{{ user.emailAddress }}</td>
+                <td><small>{{ user.emailAddress }}</small></td>
                 {% if organization.parent is null %}
                     <td>{{ user.organization.name }}</td>
                 {% endif %}

templates/organization/user/user-list.html.twig

@@ -5,6 +5,8 @@
     {{ encore_entry_script_tags('availabilitable-list') }}
 {% endblock %}
 
+{% block title %}{{ 'organization.users' | trans }}{% endblock %}
+
 {% block body %}
     <h1>{{ 'organization.users' | trans }} - {{ organization }}</h1>
 

translations/messages.fr.yaml

@@ -87,7 +87,7 @@ organization:
   userAvailabilityNextWeek: Afficher les disponibilités de mes bénévoles pour la semaine prochaine
   userList: Liste des bénévoles
   user:
-    hasDrivingLicence: Permis depuis plus de 3 ans ?
+    hasDrivingLicence: Permis B depuis plus de 3 ans ?
     isFullyEquipped: Uniforme en dotation ?
     occupation: Fonction de cadre au sein de la structure d'emploi
     skillset: Compétences Croix-Rouge
@@ -106,7 +106,7 @@ user:
   availabilitySpecificWeek: Mes disponibilités pour la semaine du %week%
   createMyAccount: Créer mon compte
   detail:
-    drivingLicence: Avez-vous le permis depuis plus de 3 ans ?
+    drivingLicence: Avez-vous le permis B depuis plus de 3 ans ?
     fullyEquipped: Avez-vous un uniforme en dotation chez vous ?
     occupation: Quelle est votre profession ?
     organization: Votre structure de rattachement