improve duplicate ux

src/Bundle/ChillPersonBundle/Resources/views/PersonDuplicate/confirm.html.twig

@@ -90,7 +90,7 @@
             </a>
         </li>
         <li>
-            <button class="sc-button bt-save" type="submit">{{ 'Confirm'|trans }}</button>
+            <button class="sc-button bt-submit" type="submit"><i class="fa fa-cog fa-fw"></i>{{ 'Merge'|trans }}</button>
         </li>
     </ul>
 

src/Bundle/ChillPersonBundle/Resources/views/PersonDuplicate/view.html.twig

@@ -7,10 +7,8 @@
 
 
 {% block personcontent %}
-
-
-
 <h1>{{ title|default('Person duplicate')|trans }}</h1>
+
 {% if duplicatePersons|length > 0 %}
     <p>{{ title|default('Person duplicate explained')|trans }}</p>
 
@@ -62,11 +60,12 @@
                     </li>
                     <li>
                         <a class="sc-button bt-action" href="{{ path('chill_person_duplicate_confirm', { person1_id : person.id, person2_id : duplicatePerson.id }) }}">
-                        {{ 'Merge'|trans }}</a>
+                        <i class="fa fa-cog fa-fw"></i>{{ 'Merge'|trans }}</a>
                     </li>
                     <li>
-                        <a class="sc-button bt-not-duplicate" href="{{ path('chill_person_duplicate_not_duplicate', {person1_id : person.id, person2_id : duplicatePerson.id}) }}">
-                        {{ 'Switch to truefalse'|trans }}</a>
+                        <a class="sc-button" title="{{ 'Switch to truefalse'|trans }}"
+                           href="{{ path('chill_person_duplicate_not_duplicate', {person1_id : person.id, person2_id : duplicatePerson.id}) }}">
+                        <i class="fa fa-toggle-on fa-fw"></i>{{ 'duplicate'|trans }}</a>
                     </li>
                 </ul>
             </td>
@@ -126,8 +125,9 @@
                             <a class="sc-button bt-show" target="_blank" href="{{ path('chill_person_view', { person_id : notDuplicatePerson.id }) }}"></a>
                         </li>
                         <li>
-                            <a class="sc-button bt-not-duplicate" href="{{ path('chill_person_remove_duplicate_not_duplicate', {person1_id : person.id, person2_id : notDuplicatePerson.id}) }}">
-                            {{ 'Switch to duplicate'|trans }}</a>
+                            <a class="sc-button" title="{{ 'Switch to duplicate'|trans }}"
+                               href="{{ path('chill_person_remove_duplicate_not_duplicate', {person1_id : person.id, person2_id : notDuplicatePerson.id}) }}">
+                            <i class="fa fa-toggle-off fa-fw"></i>{{ 'not-duplicate'|trans }}</a>
                         </li>
                     </ul>
                 </td>
@@ -135,7 +135,11 @@
         {% endfor %}
     </table>
 {% endif %}
-
+    
+{% if notDuplicatePersons|length == 0 and duplicatePersons|length == 0 %}
+    <span class="chill-no-data-statement">{{ 'No duplicate candidates'|trans }}</span>
+{% endif %}
+    
 <ul class="record_actions">
     <li class="cancel">
         <a href="{{ path('chill_person_view', {person_id: person.id }) }}" class="sc-button">

src/Bundle/ChillPersonBundle/translations/messages.fr.yml

@@ -73,12 +73,13 @@ Old person explain: sera supprimé lors de la fusion
 New person: Dossier cible
 New person explain: sera conservé lors de la fusion
 I confirm the merger of these 2 people : Je confime la fusion de ces 2 dossiers
-Person duplicate explained: Chill a détecté des doublons potentiels ! Vous pouvez confirmer, infirmer, ou encore désigner un autre doublon.
+Person duplicate explained: Chill a détecté des doublons potentiels ! Vous pouvez confirmer, infirmer, ou encore désigner manuellement un autre doublon.
 Person flaged as duplicate: Dossiers marqués comme faux-positif
 Person flaged as duplicate explained: Les dossiers suivants sont marqués comme faux-positifs. Ce ne sont pas des doublons !
-Associate manually a duplicate person: Désigner un autre doublon
+Associate manually a duplicate person: Désigner manuellement un doublon
 Invert: Inverser le sens de la fusion
 Find duplicate: Trouver un doublon
+Person duplicate: Traiter les doublons
 Open in another window: Ouvrir dans une nouvelle fenêtre
 Deleted datas: Données supprimées
 Keeped datas: Données conservées
@@ -86,8 +87,11 @@ Moved links: Relations déplacées
 Keeped links: Relations conservées
 Merge duplicate persons folders: Fusion de dossiers
 Merge: Fusionner
-Switch to truefalse: Passer en Faux-positif
-Switch to duplicate: Passer en doublon
+duplicate: Doublon
+not-duplicate: Faux-positif
+Switch to truefalse: Marquer comme faux-positif
+Switch to duplicate: Marquer comme doublon
+No duplicate candidates: Il n'y a pas de doublons détectés, ni de faux-positifs
 
 # addresses part
 address_street_address_1: Adresse ligne 1
@@ -141,7 +145,6 @@ Update accompanying period: Mettre à jour une période d'accompagnement
 "Period not opened : form is invalid": "La période n'a pas été ouverte: le formulaire est invalide."
 'Closing motive': 'Motif de clôture'
 'Person details': 'Détails de la personne'
-'Person duplicate': 'Trouver des doublons'
 'Update details for %name%': 'Modifier détails de %name%'
 Any accompanying periods are open: Aucune période d'accompagnement ouverte
 An accompanying period is open: Une période d'accompagnement est ouverte