Extract onclick handlers into script files

These are the handlers for deleting files and reverting changes.
acant · Jul 29, 2014 · d4adb4a · d4adb4a
1 parent 336ce93
commit d4adb4a
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/lib/gitdocs/public/js/app.js b/lib/gitdocs/public/js/app.js
@@ -110,3 +110,13 @@ $('a[data-method]').live('click', function(e) {
   form.hide().append(metadata_input).appendTo('body');
   form.submit();
 });
+
+// Deletion tab
+$('ul.tabs a.delete').live('click', function(e) {
+  return confirm('Are you sure?')
+});
+
+// Revert a revision from the revision table
+$('#revisions td.revert a').live('click', function(e) {
+  return confirm('Are you sure?')
+});
diff --git a/lib/gitdocs/views/_header.haml b/lib/gitdocs/views/_header.haml
@@ -11,4 +11,4 @@
     %li
       %a{ href: '?mode=revisions' } Revisions
     %li
-      %a{ href: '?mode=delete', onclick: "javascript:return confirm('Are you sure?')" } Delete
+      %a.delete{ href: '?mode=delete' } Delete
diff --git a/lib/gitdocs/views/revisions.haml b/lib/gitdocs/views/revisions.haml
@@ -22,7 +22,7 @@
           %td.author= r[:author]
           %td.date.reldate= r[:date].iso8601
           %td.revert
-            %a{ href: "?mode=revert&revision=#{r[:commit]}", onclick: "javascript:return confirm('Are you sure?')" }
+            %a{ href: "?mode=revert&revision=#{r[:commit]}" }
               = r[:commit]
 - if revisions.empty?
   %p No revisions for this file could be found.