MDL-67277 forum: Replace the HTML comments with Mustache comments

It does not seem necessary to have the actual HTML sent to browsers full of these developer-focused comments.
moodle · Dec 3, 2019 · 83e5983 · 83e5983
1 parent 0ddd9c5
commit 83e5983
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/mod/forum/templates/discussion_list.mustache b/mod/forum/templates/discussion_list.mustache
@@ -189,7 +189,7 @@
                 {{$discussion_list_body}}
                 <tbody>
                     {{#summaries}}
-                        <!-- The discussion class is only required for behat tests to identify the row -->
+                        {{! The discussion class is only required for behat tests to identify the row. }}
                         <tr class="discussion{{!
                                 }}{{#discussion.pinned}} pinned{{/discussion.pinned}}{{!
                                 }}{{#discussion.userstate.favourited}} favourited{{/discussion.userstate.favourited}}{{!

diff --git a/mod/forum/templates/forum_discussion_nested_v2_first_post.mustache b/mod/forum/templates/forum_discussion_nested_v2_first_post.mustache
@@ -39,7 +39,7 @@
     aria-labelledby="post-header-{{id}}"
     aria-describedby="post-content-{{id}}"
 >
-    <!-- The firstpost and starter classes below aren't used for anything other than to identify the first post in behat -->
+    {{! The firstpost and starter classes below aren't used for anything other than to identify the first post in behat. }}
     <div
         class="d-flex focus-target mb-4 {{#firstpost}}firstpost starter{{/firstpost}}"
         aria-label='{{#str}} postbyuser, mod_forum, {"post": "{{subject}}", "user": "{{author.fullname}}"} {{/str}}'

diff --git a/mod/forum/templates/forum_discussion_post.mustache b/mod/forum/templates/forum_discussion_post.mustache
@@ -39,7 +39,7 @@
     aria-labelledby="post-header-{{id}}"
     aria-describedby="post-content-{{id}}"
 >
-    <!-- The firstpost and starter classes below aren't used for anything other than to identify the first post in behat -->
+    {{! The firstpost and starter classes below aren't used for anything other than to identify the first post in behat. }}
     <div
         class="d-flex border p-2 mb-2 forumpost focus-target {{#unread}}unread{{/unread}} {{#firstpost}}firstpost starter{{/firstpost}}"
         aria-label='{{#str}} postbyuser, mod_forum, {"post": "{{subject}}", "user": "{{author.fullname}}"} {{/str}}'

diff --git a/mod/forum/templates/forum_discussion_threaded_posts.mustache b/mod/forum/templates/forum_discussion_threaded_posts.mustache
@@ -33,7 +33,7 @@
 {{#posts}}
     {{< mod_forum/forum_discussion_post }}
         {{$replies}}
-            <!-- The forumthread class is only added for behat -->
+            {{! The forumthread class is only added for behat. }}
             <div class="indent forumthread post-replies" data-region="replies-container">
                 {{#replies}}
                     {{> mod_forum/forum_discussion_threaded_post }}