Split the forum view into a separate .js file. We will do this for all views.
authorArt Cancro <ajc@citadel.org>
Sat, 6 Nov 2021 20:43:05 +0000 (16:43 -0400)
committerArt Cancro <ajc@citadel.org>
Sat, 6 Nov 2021 20:43:05 +0000 (16:43 -0400)
webcit-ng/static/index.html
webcit-ng/static/js/view_forum.js [new file with mode: 0644]
webcit-ng/static/js/views.js

index 81b45b27e6524e60a01c1a7e1048085f29d41070..b474124779ce51cbb86107eeb0ec2bf3150d649f 100644 (file)
@@ -1,9 +1,9 @@
 <!DOCTYPE html>
 
 <!-- 
----- Copyright (c) 1996-2019 by Art Cancro and the citadel.org team.
+---- Copyright (c) 1996-2021 by Art Cancro and the citadel.org team.
 ---- This program is open source software.  You can redistribute it and/or
----- modify it under the terms of the GNU General Public License, version 3.
+---- modify it under the terms of the GPL (General Public License) version 3.
 -->
 
 <html>
@@ -77,30 +77,37 @@ Loading...
 <script type="text/javascript" src="js/login.js"></script>
 <script type="text/javascript" src="js/main.js"></script>
 <script type="text/javascript" src="js/views.js"></script>
+<script type="text/javascript" src="js/view_forum.js"></script>
 <script>
+
 // Get the Sidebar
 var sidebar = document.getElementById("sidebar");
 
+
 // Get the DIV with overlay effect
 var overlayBg = document.getElementById("myOverlay");
 
+
 // Toggle between showing and hiding the sidebar, and add overlay effect
 function w3_open() {
        if (sidebar.style.display === 'block') {
                sidebar.style.display = 'none';
                overlayBg.style.display = "none";
-       } else {
+       }
+       else {
                sidebar.style.display = 'block';
                overlayBg.style.display = "block";
        }
 }
 
+
 // Close the sidebar with the close button
 function w3_close() {
        sidebar.style.display = "none";
        overlayBg.style.display = "none";
 }
 
+
 ctdl_startup();
 </script>
 
diff --git a/webcit-ng/static/js/view_forum.js b/webcit-ng/static/js/view_forum.js
new file mode 100644 (file)
index 0000000..4e0395d
--- /dev/null
@@ -0,0 +1,181 @@
+//
+// Copyright (c) 2016-2021 by the citadel.org team
+//
+// This program is open source software.  It runs great on the
+// Linux operating system (and probably elsewhere).  You can use,
+// copy, and run it under the terms of the GNU General Public
+// License version 3.  Richard Stallman is an asshole communist.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+
+
+// Forum view (flat) -- all rendering is done client-side
+//
+function forum_readmessages(target_div, gt_msg, lt_msg) {
+       original_text = document.getElementById(target_div).innerHTML;          // in case we need to replace it after an error
+       document.getElementById(target_div).innerHTML = 
+               "<div class=\"ctdl-forum-nav\"><i class=\"fas fa-spinner fa-spin\"></i>&nbsp;&nbsp;"
+               + _("Loading messages from server, please wait") + "</div>";
+
+       if (lt_msg < 9999999999) {
+               url = "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.lt|" + lt_msg;
+       }
+       else {
+               url = "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.gt|" + gt_msg;
+       }
+
+       fetch_msg_list = async() => {
+               response = await fetch(url);
+               msgs = await(response.json());
+               if (response.ok) {
+                       document.getElementById(target_div).innerHTML = "" ;
+
+                       // If we were given an explicit starting point, by all means start there.
+                       // Note that we don't have to remove them from the array because we did a 'msgs gt|xxx' command to Citadel.
+                       if (gt_msg > 0) {
+                               msgs = msgs.slice(0, messages_per_page);
+                       }
+
+                       // Otherwise, show us the last 20 messages
+                       else {
+                               if (msgs.length > messages_per_page) {
+                                       msgs = msgs.slice(msgs.length - messages_per_page);
+                               }
+                               new_old_div_name = randomString(5);
+                               if (msgs.length < 1) {
+                                       newlt = lt_msg;
+                               }
+                               else {
+                                       newlt = msgs[0];
+                               }
+                               document.getElementById(target_div).innerHTML +=
+                                       "<div id=\"" + new_old_div_name + "\">" +
+                                       "<div class=\"ctdl-forum-nav\">" +
+                                       "<a href=\"javascript:forum_readmessages('" + new_old_div_name + "', 0, " + newlt + ");\">" +
+                                       "<i class=\"fa fa-arrow-circle-up\"></i>&nbsp;&nbsp;" +
+                                       _("Older posts") + "&nbsp;&nbsp;<i class=\"fa fa-arrow-circle-up\"></a></div></div></a></div></div>" ;
+                       }
+
+                       // Render an empty div for each message.  We will fill them in later.
+                       for (var i in msgs) {
+                               document.getElementById(target_div).innerHTML += "<div id=\"ctdl_msg_" + msgs[i] + "\"> </div>" ;
+                               document.getElementById("ctdl_msg_"+msgs[i]).style.display = "none";
+                       }
+                       if (lt_msg == 9999999999) {
+                               new_new_div_name = randomString(5);
+                               if (msgs.length <= 0) {
+                                       newgt = gt_msg;
+                               }
+                               else {
+                                       newgt = msgs[msgs.length-1];
+                               }
+                               document.getElementById(target_div).innerHTML +=
+                                       "<div id=\"" + new_new_div_name + "\">" +
+                                       "<div class=\"ctdl-forum-nav\">" +
+                                       "<a href=\"javascript:forum_readmessages('" + new_new_div_name + "', " + newgt + ", 9999999999);\">" +
+                                       "<i class=\"fa fa-arrow-circle-down\"></i>&nbsp;&nbsp;" +
+                                       _("Newer posts") + "&nbsp;&nbsp;<i class=\"fa fa-arrow-circle-down\"></a></div></div>" ;
+                       }
+
+                       // Now figure out where to scroll to after rendering.
+                       if (gt_msg > 0) {
+                               scroll_to = msgs[0];
+                       }
+                       else if (lt_msg < 9999999999) {
+                               scroll_to = msgs[msgs.length-1];
+                       }
+                       else if ( (logged_in) && (gt_msg == 0) && (lt_msg == 9999999999) ) {
+                               scroll_to = msgs[msgs.length-1];
+                       }
+                       else {
+                               scroll_to = msgs[0];            // FIXME this is too naive
+                       }
+
+                       // Render the individual messages in the divs
+                       forum_render_messages(msgs, "ctdl_msg_", scroll_to)
+               }
+               else {
+                       // if xhr fails, this will make the link reappear so the user can try again
+                       document.getElementById(target_div).innerHTML = original_text;
+               }
+       }
+       fetch_msg_list();
+}
+
+
+// Render a range of messages, with the div prefix specified
+//
+function forum_render_messages(msgs, prefix, scroll_to) {
+       for (i=0; i<msgs.length; ++i) {
+               forum_render_one(prefix+msgs[i], msgs[i], scroll_to);
+       }
+}
+
+
+// We have to put each XHR for forum_render_messages() into its own stack frame, otherwise it jumbles them together.  I don't know why.
+function forum_render_one(div, msgnum, scroll_to) {
+       fetch_message = async() => {
+               response = await fetch("/ctdl/r/" + escapeHTMLURI(current_room) + "/" + msgs[i] + "/json");
+               msg = await response.json();
+               if (response.ok) {
+                       outmsg =
+                         "<div class=\"ctdl-msg-wrapper\">"                            // begin message wrapper
+                       + "<div class=\"ctdl-avatar\">"                                 // begin avatar
+                       + "<img src=\"/ctdl/u/" + msg.from + "/userpic\" width=\"32\" "
+                       + "onerror=\"this.parentNode.innerHTML='&lt;i class=&quot;fa fa-user-circle fa-2x&quot;&gt;&lt;/i&gt; '\">"
+                       + "</div>"                                                      // end avatar
+                       + "<div class=\"ctdl-msg-content\">"                            // begin content
+                       + "<div class=\"ctdl-msg-header\">"                             // begin header
+                       + "<span class=\"ctdl-msg-header-info\">"                       // begin header info on left side
+                       + "<span class=\"ctdl-username\"><a href=\"#\">"                // FIXME link to user profile
+                       + msg.from
+                       + "</a></span>"                                                 // end username
+                       + "<span class=\"ctdl-msgdate\">"
+                       + msg.time
+                       + "</span>"                                                     // end msgdate
+                       + "</span>"                                                     // end header info on left side
+                       + "<span class=\"ctdl-msg-header-buttons\">"                    // begin buttons on right side
+
+                       + "<span class=\"ctdl-msg-button\"><a href=\"#\">"              // Reply button
+                       + "<i class=\"fa fa-reply\"></i> " 
+                       + _("Reply")
+                       + "</a></span>"
+
+                       + "<span class=\"ctdl-msg-button\"><a href=\"#\">"              // ReplyQuoted , only show in forums
+                       + "<i class=\"fa fa-reply\"></i> " 
+                       + _("ReplyQuoted")
+                       + "</a></span>"
+
+                       + "<span class=\"ctdl-msg-button\"><a href=\"#\">"              // Delete , show only with permission
+                       + "<i class=\"fa fa-trash\"></i> " 
+                       + _("Delete")
+                       + "</a></span>"
+
+                       + "</span>";                                                    // end buttons on right side
+                       if (msg.subj) {
+                               outmsg +=
+                               "<br><span class=\"ctdl-msgsubject\">" + msg.subj + "</span>";
+                       }
+                       outmsg +=
+                         "</div><br>"                                                  // end header
+                       + "<div class=\"ctdl-msg-body\">"                               // begin body
+                       + msg.text
+                       + "</div>"                                                      // end body
+                       + "</div>"                                                      // end content
+                       + "</div>"                                                      // end wrapper
+                       ;
+                       document.getElementById(div).innerHTML = outmsg;
+               }
+               else {
+                       document.getElementById(div).innerHTML = "ERROR";
+               }
+               document.getElementById(div).style.display  = "inline";
+               if (msgnum == scroll_to) {
+                       window.location.hash = div;
+               }
+       }
+       fetch_message();
+}
index 488f850fd761d0c5b92fb14ed8a25a64d9bfdbeb..ba7bf716f179aaf3d97bf4d24f9be041deaa2308 100644 (file)
@@ -1,5 +1,5 @@
 //
-// Copyright (c) 2016-2020 by the citadel.org team
+// Copyright (c) 2016-2021 by the citadel.org team
 //
 // This program is open source software.  It runs great on the
 // Linux operating system (and probably elsewhere).  You can use,
@@ -48,172 +48,3 @@ function render_room_view(gt_msg, lt_msg) {
        }
 
 }
-
-
-// Forum view (flat) -- all rendering is done client-side
-//
-function forum_readmessages(target_div, gt_msg, lt_msg) {
-       original_text = document.getElementById(target_div).innerHTML;          // in case we need to replace it after an error
-       document.getElementById(target_div).innerHTML = 
-               "<div class=\"ctdl-forum-nav\"><i class=\"fas fa-spinner fa-spin\"></i>&nbsp;&nbsp;"
-               + _("Loading messages from server, please wait") + "</div>";
-
-       if (lt_msg < 9999999999) {
-               url = "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.lt|" + lt_msg;
-       }
-       else {
-               url = "/ctdl/r/" + escapeHTMLURI(current_room) + "/msgs.gt|" + gt_msg;
-       }
-
-       fetch_msg_list = async() => {
-               response = await fetch(url);
-               msgs = await(response.json());
-               if (response.ok) {
-                       document.getElementById(target_div).innerHTML = "" ;
-
-                       // If we were given an explicit starting point, by all means start there.
-                       // Note that we don't have to remove them from the array because we did a 'msgs gt|xxx' command to Citadel.
-                       if (gt_msg > 0) {
-                               msgs = msgs.slice(0, messages_per_page);
-                       }
-
-                       // Otherwise, show us the last 20 messages
-                       else {
-                               if (msgs.length > messages_per_page) {
-                                       msgs = msgs.slice(msgs.length - messages_per_page);
-                               }
-                               new_old_div_name = randomString(5);
-                               if (msgs.length < 1) {
-                                       newlt = lt_msg;
-                               }
-                               else {
-                                       newlt = msgs[0];
-                               }
-                               document.getElementById(target_div).innerHTML +=
-                                       "<div id=\"" + new_old_div_name + "\">" +
-                                       "<div class=\"ctdl-forum-nav\">" +
-                                       "<a href=\"javascript:forum_readmessages('" + new_old_div_name + "', 0, " + newlt + ");\">" +
-                                       "<i class=\"fa fa-arrow-circle-up\"></i>&nbsp;&nbsp;" +
-                                       _("Older posts") + "&nbsp;&nbsp;<i class=\"fa fa-arrow-circle-up\"></a></div></div></a></div></div>" ;
-                       }
-
-                       // Render an empty div for each message.  We will fill them in later.
-                       for (var i in msgs) {
-                               document.getElementById(target_div).innerHTML += "<div id=\"ctdl_msg_" + msgs[i] + "\"> </div>" ;
-                               document.getElementById("ctdl_msg_"+msgs[i]).style.display = "none";
-                       }
-                       if (lt_msg == 9999999999) {
-                               new_new_div_name = randomString(5);
-                               if (msgs.length <= 0) {
-                                       newgt = gt_msg;
-                               }
-                               else {
-                                       newgt = msgs[msgs.length-1];
-                               }
-                               document.getElementById(target_div).innerHTML +=
-                                       "<div id=\"" + new_new_div_name + "\">" +
-                                       "<div class=\"ctdl-forum-nav\">" +
-                                       "<a href=\"javascript:forum_readmessages('" + new_new_div_name + "', " + newgt + ", 9999999999);\">" +
-                                       "<i class=\"fa fa-arrow-circle-down\"></i>&nbsp;&nbsp;" +
-                                       _("Newer posts") + "&nbsp;&nbsp;<i class=\"fa fa-arrow-circle-down\"></a></div></div>" ;
-                       }
-
-                       // Now figure out where to scroll to after rendering.
-                       if (gt_msg > 0) {
-                               scroll_to = msgs[0];
-                       }
-                       else if (lt_msg < 9999999999) {
-                               scroll_to = msgs[msgs.length-1];
-                       }
-                       else if ( (logged_in) && (gt_msg == 0) && (lt_msg == 9999999999) ) {
-                               scroll_to = msgs[msgs.length-1];
-                       }
-                       else {
-                               scroll_to = msgs[0];            // FIXME this is too naive
-                       }
-
-                       // Render the individual messages in the divs
-                       forum_render_messages(msgs, "ctdl_msg_", scroll_to)
-               }
-               else {
-                       // if xhr fails, this will make the link reappear so the user can try again
-                       document.getElementById(target_div).innerHTML = original_text;
-               }
-       }
-       fetch_msg_list();
-}
-
-
-// Render a range of messages, with the div prefix specified
-//
-function forum_render_messages(msgs, prefix, scroll_to) {
-       for (i=0; i<msgs.length; ++i) {
-               forum_render_one(prefix+msgs[i], msgs[i], scroll_to);
-       }
-}
-
-
-// We have to put each XHR for forum_render_messages() into its own stack frame, otherwise it jumbles them together.  I don't know why.
-function forum_render_one(div, msgnum, scroll_to) {
-       fetch_message = async() => {
-               response = await fetch("/ctdl/r/" + escapeHTMLURI(current_room) + "/" + msgs[i] + "/json");
-               msg = await response.json();
-               if (response.ok) {
-                       outmsg =
-                         "<div class=\"ctdl-msg-wrapper\">"                            // begin message wrapper
-                       + "<div class=\"ctdl-avatar\">"                                 // begin avatar
-                       + "<img src=\"/ctdl/u/" + msg.from + "/userpic\" width=\"32\" "
-                       + "onerror=\"this.parentNode.innerHTML='&lt;i class=&quot;fa fa-user-circle fa-2x&quot;&gt;&lt;/i&gt; '\">"
-                       + "</div>"                                                      // end avatar
-                       + "<div class=\"ctdl-msg-content\">"                            // begin content
-                       + "<div class=\"ctdl-msg-header\">"                             // begin header
-                       + "<span class=\"ctdl-msg-header-info\">"                       // begin header info on left side
-                       + "<span class=\"ctdl-username\"><a href=\"#\">"                // FIXME link to user profile
-                       + msg.from
-                       + "</a></span>"                                                 // end username
-                       + "<span class=\"ctdl-msgdate\">"
-                       + msg.time
-                       + "</span>"                                                     // end msgdate
-                       + "</span>"                                                     // end header info on left side
-                       + "<span class=\"ctdl-msg-header-buttons\">"                    // begin buttons on right side
-
-                       + "<span class=\"ctdl-msg-button\"><a href=\"#\">"              // Reply button
-                       + "<i class=\"fa fa-reply\"></i> " 
-                       + _("Reply")
-                       + "</a></span>"
-
-                       + "<span class=\"ctdl-msg-button\"><a href=\"#\">"              // ReplyQuoted , only show in forums
-                       + "<i class=\"fa fa-reply\"></i> " 
-                       + _("ReplyQuoted")
-                       + "</a></span>"
-
-                       + "<span class=\"ctdl-msg-button\"><a href=\"#\">"              // Delete , show only with permission
-                       + "<i class=\"fa fa-trash\"></i> " 
-                       + _("Delete")
-                       + "</a></span>"
-
-                       + "</span>";                                                    // end buttons on right side
-                       if (msg.subj) {
-                               outmsg +=
-                               "<br><span class=\"ctdl-msgsubject\">" + msg.subj + "</span>";
-                       }
-                       outmsg +=
-                         "</div><br>"                                                  // end header
-                       + "<div class=\"ctdl-msg-body\">"                               // begin body
-                       + msg.text
-                       + "</div>"                                                      // end body
-                       + "</div>"                                                      // end content
-                       + "</div>"                                                      // end wrapper
-                       ;
-                       document.getElementById(div).innerHTML = outmsg;
-               }
-               else {
-                       document.getElementById(div).innerHTML = "ERROR";
-               }
-               document.getElementById(div).style.display  = "inline";
-               if (msgnum == scroll_to) {
-                       window.location.hash = div;
-               }
-       }
-       fetch_message();
-}