Merge branch '1706022104-133' into 'master'

[#133] QA: Fix Bugs Detected By Sonarqube (Part 2)

Closes #133

See merge request !99

administration/templates/detail_verif.html

@@ -52,25 +52,31 @@ Pratinjau Materi
                 {% if riwayat %}
                 <div class="info-wrapper">
                     <div class="info" id="1">
-                        <dt class="col col-4">
-                            <p class="info-name"><strong>Verifikatur</strong></p>
-                        </dt>
+                        <dl>
+                            <dt class="col col-4">
+                                <p class="info-name"><strong>Verifikatur</strong></p>
+                            </dt>
+                        </dl>
                         <dd>
                             <p class="info-content">{{verification_report.user.name}}</p>
                         </dd>
                     </div>
                     <div class="info" id="1">
-                        <dt class="col col-4">
-                            <p class="info-name"><strong>Waktu Verifikasi</strong></p>
-                        </dt>
+                        <dl>
+                            <dt class="col col-4">
+                                <p class="info-name"><strong>Waktu Verifikasi</strong></p>
+                            </dt>
+                        </dl>
                         <dd>
                             <p class="info-content">{{verification_report.timestamp}}</p>
                         </dd>
                     </div>
                     <div class="info" id="1">
-                        <dt class="col col-4">
-                            <p class="info-name"><strong>Status Materi</strong></p>
-                        </dt>
+                        <dl>
+                            <dt class="col col-4">
+                                <p class="info-name"><strong>Status Materi</strong></p>
+                            </dt>
+                        </dl>
                         <dd>
                             <p class="info-content">{{verification_report.status}}</p>
                         </dd>

administration/templates/laporan_materi.html

@@ -22,6 +22,7 @@
   <div class="card-body">
     <div class="table-responsive">
       <table class="table table-bordered" id="dataTable">
+        <caption>Materi yang Dilaporkan</caption>
         <thead>
           <tr>
             <th scope="col">Judul</th>

app/templates/app/detail_materi.html

@@ -312,7 +312,7 @@ div.review {
                 <div style="border-top:1px solid #d4d4d4;text-align:center">
                     <h3>Video</h3>
                     <iframe src="https://www.youtube.com/embed/{{ materi_data.yt_video_id }}"
-                    allowfullscreen="allowfullscreen"
+                    title="Video materi" allowfullscreen="allowfullscreen"
                     width="564" height="300">
                         invalid video url
                     </iframe>
@@ -472,7 +472,7 @@ div.review {
 {% endblock content %}
 {% block extra_scripts %}
 <script src="https://kit.fontawesome.com/bc2cedd6b2.js" crossorigin="anonymous"></script>
-<script type="text/javascript">
+<script>
     // using jQuery
     var csrftoken = jQuery("[name=csrfmiddlewaretoken]").val();
 

app/templates/statistik.html

@@ -24,9 +24,10 @@
     <div class="card-body">
         <div class="table-responsive">
             <table class="table table-bordered">
+                <caption>Summary Materi per Kategori</caption>
                 <thead>
-                    <th>Kategori</th>
-                    <th>Jumlah Materi</th>
+                    <th scope="col">Kategori</th>
+                    <th scope="col">Jumlah Materi</th>
                 </thead>
                 <tbody>
                     {% for s in stats %}