make module prereq more accessible

fixes CNVS-13550

-- Test Plan --

As a user that can edit module items
Given I'm using JAWS and/or voice over
When I edit a module
And I add a pre-req to the module
Then adding a pre-req should be accessible in JAWs
And the "remove" pre-req X should be a button
And the "remove" pre-req X should have a label on it

Change-Id: I421134cbf42f03bac9d7a9a9bc3d425937f7374b
Reviewed-on: https://gerrit.instructure.com/36290
Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
QA-Review: Clare Strong <clare@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
Product-Review: Hilary Scharton <hilary@instructure.com>

app/stylesheets/g_wiki.sass

@@ -52,6 +52,22 @@ $revision-button-hover-color: #2590cf
   #breadcrumbs
     :border-bottom $pages-border
 
+.delete_criterion_link
+  border: none
+  background: none
+  padding: 0px
+
+
+.add_prerequisite_link
+  color: #0088cc
+  border: none
+  background: none
+  padding: 0px
+
+  &:hover
+    color: #005580
+    text-decoration: underline
+ 
 .pages
   .header-bar-outer-container
     :height 64px

app/views/context_modules/_content_next.html.erb

@@ -223,7 +223,9 @@ TEXT
             <%= f.text_field :name, :placeholder => t('module_name', %{Module Name}), :class => 'input-block-level' %>
           </div>
         </td>
-      </tr><tr>
+      </tr>
+
+      <tr>
         <td colspan="2">
           <input type="checkbox" id="unlock_module_at"/>
           <label for="unlock_module_at"><%= t('#labels.lock_module_until', %{Lock module until a given date}) %></label>
@@ -234,7 +236,9 @@ TEXT
         </td><td>
           <%= f.text_field :unlock_at, :class => "datetime_field", :'aria-label' => t('#aria_labels.unlock_at_date', "Unlock at date") %>
         </td>
-      </tr><tr>
+      </tr>
+
+      <tr>
         <td colspan="2">
         <div class="prerequisites_entry">
           <b><%= before_label('modules.before_students_view', %{Before students can view this module}) %></b>
@@ -249,16 +253,16 @@ TEXT
                 </span>
               </div>
               <div style="float: right;">
-                <a href="#" class="delete_criterion_link dim-till-hover"><i class="icon-end standalone-icon" title="Delete"></i></a>
+                <button class="delete_criterion_link dim-till-hover" aria-label="<%= t('button.delete_label', %{Delete prerequisites}) %>" title="<%= t('button.delete_title', %{Delete prerequisites}) %>"><i class="icon-end standalone-icon" ></i></button>
               </div>
               <div class="clear"></div>
             </div>
             <div style="font-size: 0.9em;">
-              <a href="#" class="add_prerequisite_link icon-add"><%= t('links.add_prerequisite', %{Add prerequisite}) %></a>
+              <button class="add_prerequisite_link"><i class="icon-add"></i><%= t('links.add_prerequisite', %{Add prerequisite}) %></button>
             </div>
           </div>
         </div>
-        <div class="completion_entry">
+        <div class="completion_entry" style="margin-top:10px;">
           <b><%= before_label('module_complete_when', %{This module is complete when}) %></b>
           <div class="completion_criteria_list">
             <div class="no_items_message"><%= t('prerequisites_need_item', %{You'll need to add items before you can specify how a user will complete this module}) %></div>
@@ -270,8 +274,9 @@ TEXT
             </div>
           </div>
         </div>
-      </td>
+       </td>
-    </tr><tr>
+      </tr>
+     <tr>
         <td colspan="2">
           <%= f.check_box :require_sequential_progress, :id => "require_sequential_progress" %>
           <%= f.label :require_sequential_progress, :en => "Students must move through requirements in this module in sequential order", :for => "require_sequential_progress" %>