]> git.donarmstrong.com Git - debbugs.git/commitdiff
projects / debbugs.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: e71013e)
Use a checkbox and CSS to show info messages instead of JavaScript
authorPaul Wise <pabs@debian.org>
Sun, 13 Nov 2022 03:09:43 +0000 (11:09 +0800)
committerDon Armstrong <don@donarmstrong.com>
Sun, 4 Dec 2022 23:33:09 +0000 (15:33 -0800)
This works in browsers where users have disabled JavaScript,
such as the Tor Browser in the Security Level called Safest.

Obsoletes: https://salsa.debian.org/debbugs-team/debbugs/-/merge_requests/9

html/bugs.css patch | blob | history
templates/en_US/cgi/bugreport.tx patch | blob | history

diff --git a/html/bugs.css b/html/bugs.css
index 7bddb101ded04d265c34427a374ebf0f96c4d53a..59e257a88d536a00732de7d155ec8a4927f0e062 100644 (file)
--- a/html/bugs.css
+++ b/html/bugs.css
@@ -148,6 +148,11 @@ pre.mime {
     color: #797979;
 }
 
+/* Show info messages when uselessmessages is checked */
+#uselessmessages:checked ~ .infmessage {
+    display: block;
+}
+
 /* This must be separate from the other CSS to make the showing of
    unimportant messages work in bugreport.cgi. */ 
 .infmessage { display: none; }
diff --git a/templates/en_US/cgi/bugreport.tx b/templates/en_US/cgi/bugreport.tx
index 361a3b4b4748d7e96d26be06c6b51017140f9797..eee77d6fd8edfaa28f47ba0e91f500ce58f57221 100644 (file)
--- a/templates/en_US/cgi/bugreport.tx
+++ b/templates/en_US/cgi/bugreport.tx
@@ -1,20 +1,5 @@
 <: include "html/pre_title.tx" :>#<: $bug.id :> - <: $bug.subject :> - <: $config.project :> <: $config.bug :> report logs<: include "html/post_title.tx" :>
 <link rel="canonical" href="<: $bug.url :>">
-<script type="text/javascript">
-<!--
-function toggle_infmessages()
-{
-        allDivs=document.getElementsByTagName("div");
-        for (var i = 0 ; i < allDivs.length ; i++ )
-       {
-                if (allDivs[i].className == "infmessage")
-                {
-                        allDivs[i].style.display=(allDivs[i].style.display == 'none' | allDivs[i].style.display == '') ? 'block' : 'none';
-                }
-       }
-}
--->
-</script>
 </head>
 <body>
     <div class="debbugs_install"><: $config.project :> <: $config.bug :> report logs</div>
@@ -47,9 +32,8 @@ to this <: $config.bug :>.</span>
 </div>
 : }
 <: raw($log) :>
-<script>
- document.write("<span class=\"uselessmessages\"><a href=\"javascript:toggle_infmessages();\">Toggle useless messages</a></span>")
-</script>
+<input id="uselessmessages" type="checkbox">
+<label for="uselessmessages">Display info messages</label>
 <hr>
 <p class="msgreceived">Send a report that <a href="<: $config.cgi_domain :>/bugspam.cgi?bug=<: $bug_num :>">this bug log contains spam</a>.</p>
 <hr>
Debian bug tracker development