Don’t use an i tag for icons

app/assets/stylesheets/new_design/icons.scss

@@ -1,4 +1,4 @@
-i.icon {
+.icon {
   display: inline-block;
   width: 24px;
   height: 24px;

app/views/new_gestionnaire/avis/_avis_list.html.haml

@@ -14,12 +14,12 @@
               - if avis.confidentiel?
                 %span.confidentiel
                   confidentiel
-                  %i.icon.lock{ title: "Cet avis n'est pas affiché avec les autres experts consultés" }
+                  .icon.lock{ title: "Cet avis n'est pas affiché avec les autres experts consultés" }
               %span.date Demande d'avis envoyée le #{I18n.l(avis.created_at.localtime, format: '%d/%m/%y')}
             %p= avis.introduction
 
             .answer.flex.align-start
-              %i.icon.bubble.avis-icon
+              .icon.bubble.avis-icon
               .width-100
                 %h2.gestionnaire
                   = (avis.email_to_display == current_gestionnaire.email) ? 'Vous' : avis.email_to_display

app/views/new_gestionnaire/avis/index.html.haml

@@ -29,7 +29,7 @@
             %tr
               %td.number-col
                 = link_to(avis_path(avis), class: 'cell-link') do
-                  %i.icon.folder
+                  .icon.folder
                   #{avis.dossier.id}
               %td= link_to(avis.dossier.user.email, avis_path(avis), class: 'cell-link')
               %td= link_to(avis.dossier.procedure.libelle, avis_path(avis), class: 'cell-link')

app/views/new_gestionnaire/avis/instruction.html.haml

@@ -14,7 +14,7 @@
       = f.text_area :answer, rows: 3, placeholder: 'Votre avis', required: true
       .flex.justify-between.align-baseline
         %p.confidentiel
-          %i.icon.lock
+          .icon.lock
           Cet avis est confidentiel et n'est pas affiché aux autres experts consultés
         .send-wrapper
           = f.submit 'Envoyer votre avis', class: 'button send'
@@ -29,7 +29,7 @@
       .flex.justify-between.align-baseline
         - if @avis.confidentiel?
           %p.confidentiel
-            %i.icon.lock
+            .icon.lock
             Cet avis est confidentiel et n'est pas affiché aux autres experts consultés
           .send-wrapper
             = f.submit 'Demander un avis', class: 'button send'

app/views/new_gestionnaire/dossiers/_header.html.haml

@@ -8,7 +8,7 @@
           = "Dossier nº #{dossier.id}"
       %div
         = link_to print_dossier_path(dossier.procedure, dossier), target: "_blank", class: "button icon-only" do
-          %i.icon.printer>
+          .icon.printer>
         = render partial: "new_gestionnaire/procedures/dossier_actions", locals: { procedure: dossier.procedure, dossier: dossier, dossier_is_followed: current_gestionnaire&.follow?(dossier) }
         = render partial: "state_button", locals: { dossier: dossier }
     %ul.tabs

app/views/new_gestionnaire/dossiers/_state_button.html.haml

@@ -5,13 +5,13 @@
       %ul.dropdown-items
         - if dossier.en_construction?
           %li.selected
-            %i.icon.edit
+            .icon.edit
             .description
               %h4 En construction
               Vous permettez à l'usager de modifier ses réponses au formulaire
           %li
             = link_to backoffice_dossier_receive_path(dossier, new_ui: true), method: :post, data: { confirm: "Confirmer vous le passage en instruction de ce dossier ?" } do
-              %i.icon.in-progress
+              .icon.in-progress
               .description
                 %h4 Passer en instruction
                 L'usager ne pourra plus modifier le formulaire
@@ -19,35 +19,35 @@
         - if dossier.en_instruction?
           %li
             = link_to backoffice_dossier_reopen_path(dossier, new_ui: true), method: :post, data: { confirm: "Confirmer vous la réouverture de ce dossier ?" } do
-              %i.icon.edit
+              .icon.edit
               .description
                 %h4 Repasser en construction
                 Vous permettrez à l'usager de modifier ses réponses au formulaire
           %li.selected
-            %i.icon.in-progress
+            .icon.in-progress
             .description
               %h4 En instruction
               L'usager ne peut modifer son dossier pendant l'instruction
           %li{ onclick: "TPS.acceptDossier();" }
-            %i.icon.accept
+            .icon.accept
             .description
               %h4 Accepter
               L'usager sera notifié que son dossier a été accepté
           %li
             = link_to backoffice_dossier_process_dossier_path(dossier, process_action: "without_continuation", new_ui: true), method: :post, data: { confirm: "Confirmer vous le classement sans suite de ce dossier ?" } do
-              %i.icon.without-continuation
+              .icon.without-continuation
               .description
                 %h4 Classer sans suite
                 L'usager ne recevra aucune notification
           %li
             = link_to backoffice_dossier_process_dossier_path(dossier, process_action: "refuse", new_ui: true), method: :post, data: { confirm: "Confirmer vous le refus de ce dossier ?" } do
-              %i.icon.close
+              .icon.close
               .description
                 %h4 Refuser
                 L'usager sera notifié que son dossier a été refusé
       .motivation
         %h3
-          %i.icon.accept
+          .icon.accept
           Accepter le dossier
           = "nº #{dossier.id}"
 

app/views/new_gestionnaire/dossiers/messagerie.html.haml

@@ -17,7 +17,7 @@
           - if file = commentaire.piece_justificative
             .attachment-link
               = link_to file.content_url, class: "button", target: "_blank", title: "Télécharger" do
-                %i.icon.attachment
+                .icon.attachment
                 = file.original_filename
 
   = form_for(Commentaire.new, url: commentaire_dossier_path(@dossier.procedure, @dossier), html: { class: 'form' }) do |f|

app/views/new_gestionnaire/procedures/_dossier_actions.html.haml

@@ -1,19 +1,19 @@
 - if dossier.en_construction_ou_instruction?
   - if dossier_is_followed
     = link_to unfollow_dossier_path(procedure, dossier), method: :patch, class: 'button' do
-      %i.icon.unfollow>
+      .icon.unfollow>
       Ne plus suivre
   - else
     = link_to follow_dossier_path(procedure, dossier), method: :patch, class: 'button' do
-      %i.icon.follow>
+      .icon.follow>
       Suivre le dossier
 
 - elsif dossier.termine?
   - if dossier.archived
     = link_to unarchive_dossier_path(procedure, dossier), method: :patch, class: 'button' do
-      %i.icon.unarchive>
+      .icon.unarchive>
       Désarchiver le dossier
   - else
     = link_to archive_dossier_path(procedure, dossier), method: :patch, class: 'button' do
-      %i.icon.archive>
+      .icon.archive>
       Archiver le dossier

app/views/new_gestionnaire/procedures/show.html.haml

@@ -98,7 +98,7 @@
             %tr
               %td.number-col
                 = link_to(dossier_path(@procedure, dossier), class: 'cell-link') do
-                  %i.icon.folder
+                  .icon.folder
                     - if @followed_dossiers.with_unread_notifications.include?(dossier)
                       %span.notifications{ 'aria-label': 'notifications' }
                   = dossier.id

app/views/new_gestionnaire/recherches/index.html.haml

@@ -16,7 +16,7 @@
         %tr
           %td.number-col
             = link_to(dossier_path(dossier.procedure, dossier), class: 'cell-link') do
-              %i.icon.folder>
+              .icon.folder>
               = dossier.id
           %td= link_to(dossier.procedure.libelle, dossier_path(dossier.procedure, dossier), class: 'cell-link')
           %td= link_to(dossier.user.email, dossier_path(dossier.procedure, dossier), class: 'cell-link')

app/views/root/patron.html.haml

@@ -2,20 +2,20 @@
   .container
     %h1 Icones
 
-    %i.icon.follow
-    %i.icon.unfollow
-    %i.icon.archive
-    %i.icon.unarchive
-    %i.icon.folder
-    %i.icon.accept
-    %i.icon.close
-    %i.icon.without-continuation
-    %i.icon.edit
-    %i.icon.in-progress
-    %i.icon.bubble
-    %i.icon.attachment
-    %i.icon.lock
-    %i.icon.printer
+    .icon.follow
+    .icon.unfollow
+    .icon.archive
+    .icon.unarchive
+    .icon.folder
+    .icon.accept
+    .icon.close
+    .icon.without-continuation
+    .icon.edit
+    .icon.in-progress
+    .icon.bubble
+    .icon.attachment
+    .icon.lock
+    .icon.printer
 
     %h1 Formulaires
 
@@ -48,17 +48,17 @@
 
     %p
       = link_to "#", class: "button" do
-        %i.icon.follow
-        = "%i.icon.follow"
+        .icon.follow
+        = ".icon.follow"
       = link_to "#", class: "button" do
-        %i.icon.unfollow
-        = "%i.icon.unfollow"
+        .icon.unfollow
+        = ".icon.unfollow"
       = link_to "#", class: "button" do
-        %i.icon.archive
-        = "%i.icon.archive"
+        .icon.archive
+        = ".icon.archive"
       = link_to "#", class: "button" do
-        %i.icon.unarchive
-        = "%i.icon.unarchive"
+        .icon.unarchive
+        = ".icon.unarchive"
     %p
       = link_to ".button.primary.expand", "#", class: "button primary expand"
 
@@ -191,7 +191,7 @@
           %p Bonjour, merci de me donner votre avis sur ce dossier.
 
           .answer.flex.align-start
-            %i.icon.bubble.avis-icon
+            .icon.bubble.avis-icon
             .width-100
               %h2.gestionnaire
                 gestionnnaire@tps.com