Merge pull request #1082 from sgmap/fix-1056

Buttons and icons improvements

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

@@ -14,12 +14,12 @@
               - if avis.confidentiel?
                 %span.confidentiel
                   confidentiel
-                  .icon.lock{ title: "Cet avis n'est pas affiché avec les autres experts consultés" }
+                  %span.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
-              .icon.bubble.avis-icon
+              %span.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

@@ -32,7 +32,7 @@
             %tr
               %td.number-col
                 = link_to(avis_path(avis), class: 'cell-link') do
-                  .icon.folder
+                  %span.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

@@ -16,7 +16,7 @@
       = f.text_area :answer, rows: 3, placeholder: 'Votre avis', required: true
       .flex.justify-between.align-baseline
         %p.confidentiel
-          .icon.lock
+          %span.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'
@@ -31,7 +31,7 @@
       .flex.justify-between.align-baseline
         - if @avis.confidentiel?
           %p.confidentiel
-            .icon.lock
+            %span.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}"
       .mixed-buttons-bar
         .button.dropdown.icon-only.print-menu-opener{ onclick: "javascript:TPS.togglePrintMenu(event);" }
-          .icon.printer
+          %span.icon.printer
           %ul.print-menu
             %li
               = link_to "Tout le dossier", print_dossier_path(dossier.procedure, dossier), target: "_blank", class: "menu-item menu-link"

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

@@ -5,13 +5,13 @@
       %ul.dropdown-items
         - if dossier.en_construction?
           %li.selected
-            .icon.edit
+            %span.icon.edit
             .description
               %h4 En construction
               Vous permettez à l'usager de modifier ses réponses au formulaire
           %li
             = link_to passer_en_instruction_dossier_path(dossier.procedure, dossier), method: :post, data: { confirm: "Confirmer vous le passage en instruction de ce dossier ?" } do
-              .icon.in-progress
+              %span.icon.in-progress
               .description
                 %h4 Passer en instruction
                 L'usager ne pourra plus modifier le formulaire
@@ -19,27 +19,27 @@
         - if dossier.en_instruction?
           %li
             = link_to repasser_en_construction_dossier_path(dossier.procedure, dossier), method: :post, data: { confirm: "Confirmer vous le passage en construction de ce dossier ?" } do
-              .icon.edit
+              %span.icon.edit
               .description
                 %h4 Repasser en construction
                 Vous permettrez à l'usager de modifier ses réponses au formulaire
           %li.selected
-            .icon.in-progress
+            %span.icon.in-progress
             .description
               %h4 En instruction
               L'usager ne peut modifier son dossier pendant l'instruction
           %li{ onclick: "TPS.showMotivation('accept');" }
-            .icon.accept
+            %span.icon.accept
             .description
               %h4 Accepter
               L'usager sera notifié que son dossier a été accepté
           %li{ onclick: "TPS.showMotivation('without-continuation');" }
-            .icon.without-continuation
+            %span.icon.without-continuation
             .description
               %h4 Classer sans suite
               L'usager ne recevra aucune notification
           %li{ onclick: "TPS.showMotivation('refuse');" }
-            .icon.refuse
+            %span.icon.refuse
             .description
               %h4 Refuser
               L'usager sera notifié que son dossier a été refusé

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

@@ -1,6 +1,6 @@
 .motivation{ class: popup_class }
   %h3
-    .icon{ class: popup_class }
+    %span.icon{ class: popup_class }
     #{popup_title} nº #{dossier.id}
 
   = form_tag(terminer_dossier_path(dossier.procedure, dossier), method: :post, class: 'form') do

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
-      .icon.unfollow>
+      %span.icon.unfollow>
       Ne plus suivre
   - else
     = link_to follow_dossier_path(procedure, dossier), method: :patch, class: 'button' do
-      .icon.follow>
+      %span.icon.follow>
       Suivre le dossier
 
 - elsif dossier.termine?
   - if dossier.archived
     = link_to unarchive_dossier_path(procedure, dossier), method: :patch, class: 'button' do
-      .icon.unarchive>
+      %span.icon.unarchive>
       Désarchiver le dossier
   - else
     = link_to archive_dossier_path(procedure, dossier), method: :patch, class: 'button' do
-      .icon.archive>
+      %span.icon.archive>
       Archiver le dossier

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

@@ -94,7 +94,7 @@
           - @dossiers.each do |dossier|
             %tr
               %td.folder-col
-                .icon.folder
+                %span.icon.folder
                   - if current_gestionnaire.notifications_for_procedure(@procedure).include?(dossier.id)
                     %span.notifications{ 'aria-label': 'notifications' }
 

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

@@ -18,7 +18,7 @@
         %tr
           %td.number-col
             = link_to(dossier_path(dossier.procedure, dossier), class: 'cell-link') do
-              .icon.folder>
+              %span.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/new_gestionnaire/shared/commentaires/_commentaire.html.haml

@@ -12,10 +12,10 @@
   - if commentaire.piece_justificative
     .attachment-link
       = link_to commentaire.piece_justificative.content_url, class: "button", target: "_blank", title: "Télécharger" do
-        .icon.attachment
+        %span.icon.attachment
         = commentaire.piece_justificative.original_filename
   - elsif commentaire.file.present?
     .attachment-link
       = link_to commentaire.file_url, class: "button", target: "_blank", title: "Télécharger" do
-        .icon.attachment
+        %span.icon.attachment
         = commentaire.file_identifier

app/views/root/patron.html.haml

@@ -2,20 +2,20 @@
   .container
     %h1 Icones
 
-    .icon.follow
+    %span.icon.follow
-    .icon.unfollow
+    %span.icon.unfollow
-    .icon.archive
+    %span.icon.archive
-    .icon.unarchive
+    %span.icon.unarchive
-    .icon.folder
+    %span.icon.folder
-    .icon.accept
+    %span.icon.accept
-    .icon.refuse
+    %span.icon.refuse
-    .icon.without-continuation
+    %span.icon.without-continuation
-    .icon.edit
+    %span.icon.edit
-    .icon.in-progress
+    %span.icon.in-progress
-    .icon.bubble
+    %span.icon.bubble
-    .icon.attachment
+    %span.icon.attachment
-    .icon.lock
+    %span.icon.lock
-    .icon.printer
+    %span.icon.printer
 
     %h1 Formulaires
 
@@ -42,23 +42,18 @@
 
       = link_to ".button.accepted", "#", class: "button accepted"
 
+      = link_to "#", class: "button" do
+        %span.icon.follow
+        = "Bouton avec icone et texte"
+
+      = link_to "#", class: "button icon-only" do
+        %span.icon.follow
+
+    %p
       = link_to ".button.large", "#", class: "button large"
 
       = link_to ".button.large.primary", "#", class: "button large primary"
 
-    %p
-      = link_to "#", class: "button" do
-        .icon.follow
-        = ".icon.follow"
-      = link_to "#", class: "button" do
-        .icon.unfollow
-        = ".icon.unfollow"
-      = link_to "#", class: "button" do
-        .icon.archive
-        = ".icon.archive"
-      = link_to "#", class: "button" do
-        .icon.unarchive
-        = ".icon.unarchive"
     %p
       = link_to ".button.primary.expand", "#", class: "button primary expand"