Eliminate inline JavaScript, as there's no way to use it with any CSP.

marcespie · Jul 18, 2019 · 7d69537 · 7d69537
1 parent c86fc34
commit 7d69537
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 24 deletions.
diff --git a/public/javascripts/script.js b/public/javascripts/script.js
@@ -0,0 +1,20 @@
+$('#test_depends_link').click(function() {
+	$('#test-depends-list').slideToggle('fast', function() {
+		// ok
+	});
+	return(false);
+    });
+
+$('#reverse_depends_link').click(function() {
+	$('#reverse-depends-list').slideToggle('fast', function() {
+		// ok
+	});
+	return(false);
+    });
+
+$('#files_list_link').click(function() {
+	$('#files-list').slideToggle('fast', function() {
+		// ok
+	});
+	return(false);
+    });
diff --git a/views/layouts/main.tt b/views/layouts/main.tt
@@ -30,3 +30,4 @@
 <div id="footer">
 <a href="/path/databases/ports-readmes-dancer">Ports Readmes</a>, powered by <a href="http://perldancer.org/">Dancer</a> <% dancer_version %>
 </div>
+<script type="text/javascript" src="//<% request.host %>/javascripts/script.js"></script>
diff --git a/views/port.tt b/views/port.tt
@@ -99,14 +99,6 @@ on <% b.arch | html %>:
 <% END %>
 </ul>
 </div>
-<script type="text/javascript">
-$('#test_depends_link').click(function() {
-	$('#test-depends-list').slideToggle('fast', function() {
-		// ok
-	});
-	return(false);
-    });
-</script>
 <% END %>
 <%IF reversedepends %>
 <h3><a href="#" id="reverse_depends_link">Reverse dependencies</a></h3>
@@ -117,14 +109,6 @@ $('#test_depends_link').click(function() {
 <% END %>
 </ul>
 </div>
-<script type="text/javascript">
-$('#reverse_depends_link').click(function() {
-	$('#reverse-depends-list').slideToggle('fast', function() {
-		// ok
-	});
-	return(false);
-    });
-</script>
 <% END %>
 <% IF files %>
 <h3><a href="#" id="files_list_link">Files</a></h3>
@@ -135,12 +119,4 @@ $('#reverse_depends_link').click(function() {
 <% END %>
 </ul>
 </div>
-<script type="text/javascript">
-$('#files_list_link').click(function() {
-	$('#files-list').slideToggle('fast', function() {
-		// ok
-	});
-	return(false);
-    });
-</script>
 <% END %>