Minor tidying up

app/views/admin/events/_proposal.html.haml

@@ -73,9 +73,7 @@
           %td
             = link_to @event.submitter.name, admin_user_path(@event.submitter)
             - if @event.submitter.email_public
-              (
+              = "(#{mail_to(@event.submitter.email)})"
-              = link_to @event.submitter.email, "mailto: #{@event.submitter.email}"
-              )
         %tr
           %td
             %b Speakers
@@ -84,9 +82,7 @@
               %div
                 = link_to speaker.name, admin_user_path(speaker)
                 - if speaker.email_public
-                  (
+                  = "(#{mail_to(speaker.email)})"
-                  = link_to speaker.email, "mailto: #{speaker.email}"
-                  )
         %tr
           %td
             %b Volunteers
@@ -117,6 +113,7 @@
           %td
             %b Submission Description
           %td= markdown(@event.submission_text)
+
         %tr
           %td
             %b Requirements

app/views/proposals/_proposal_form.html.haml

@@ -28,9 +28,10 @@
     = render 'proposals/submission_type_content_form', f: f, program: @program
 
     -# TODO-SNAPCON: Extract this to be a `conference_committee_role?`
-    - if current_user.has_any_role? :admin, { name: :organizer, resource: @conference }, { name: :cfp, resource: @conference }
+    - read_only = !current_user.has_any_role? :admin, { name: :organizer, resource: @conference }, { name: :cfp, resource: @conference }
       %br
-      = f.input :committee_review, input_html: { rows: 5, data: { provide: 'markdown' } },
+      = f.input :committee_review,
+                input_html: { rows: 5, data: { provide: 'markdown' }, readonly: read_only },
                 hint: markdown_hint('This field is shared with the submission authors.')
 
     - if @program.cfp&.enable_registrations?