Replace accordion Javascript component by <details> HTML element

10 files changed, 30 insertions, 50 deletions

diff --git a/app/Template/project_overview/activity.php b/app/Template/project_overview/activity.php
index 2eb7a193..d3a8e55e 100644
--- a/app/Template/project_overview/activity.php
+++ b/app/Template/project_overview/activity.php
@@ -1,8 +1,6 @@
-<section class="accordion-section <?= empty($events) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Last activity') ?></h3>
-    </div>
+<details class="accordion-section"  <?= empty($events) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Last activity') ?></summary>
     <div class="accordion-content">
         <?= $this->render('event/events', array('events' => $events)) ?>
     </div>
-</section>
\ No newline at end of file
+</details>
\ No newline at end of file
diff --git a/app/Template/project_overview/attachments.php b/app/Template/project_overview/attachments.php
index b8baadd0..745b0a70 100644
--- a/app/Template/project_overview/attachments.php
+++ b/app/Template/project_overview/attachments.php
@@ -1,7 +1,5 @@
-<section class="accordion-section <?= empty($files) && empty($images) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Attachments') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($files) && empty($images) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Attachments') ?></summary>
     <div class="accordion-content">
         <?php if ($this->user->hasProjectAccess('ProjectFileController', 'create', $project['id'])): ?>
         <div class="buttons-header">
@@ -12,4 +10,4 @@
         <?= $this->render('project_overview/images', array('project' => $project, 'images' => $images)) ?>
         <?= $this->render('project_overview/files', array('project' => $project, 'files' => $files)) ?>
     </div>
-</section>
+</details>
diff --git a/app/Template/project_overview/description.php b/app/Template/project_overview/description.php
index 80b93efe..38deda95 100644
--- a/app/Template/project_overview/description.php
+++ b/app/Template/project_overview/description.php
@@ -1,7 +1,5 @@
-<section class="accordion-section <?= empty($project['description']) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Description') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($project['description']) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Description') ?></summary>
     <div class="accordion-content">
         <?php if ($this->user->hasProjectAccess('ProjectEditController', 'show', $project['id'])): ?>
             <div class="buttons-header">
@@ -12,4 +10,4 @@
             <?= $this->text->markdown($project['description']) ?>
         </article>
     </div>
-</section>
+</details>
diff --git a/app/Template/project_overview/information.php b/app/Template/project_overview/information.php
index e8c20903..13e1af01 100644
--- a/app/Template/project_overview/information.php
+++ b/app/Template/project_overview/information.php
@@ -1,7 +1,5 @@
-<section class="accordion-section">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Information') ?></h3>
-    </div>
+<details class="accordion-section" open>
+    <summary class="accordion-title"><?= t('Information') ?></summary>
     <div class="accordion-content">
         <div class="panel">
             <ul>
@@ -36,4 +34,4 @@
             </ul>
         </div>
     </div>
-</section>
+</details>
diff --git a/app/Template/subtask/show.php b/app/Template/subtask/show.php
index fe436d51..e5d3860a 100644
--- a/app/Template/subtask/show.php
+++ b/app/Template/subtask/show.php
@@ -1,7 +1,5 @@
-<section class="accordion-section <?= empty($subtasks) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Sub-Tasks') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($subtasks) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Sub-Tasks') ?></summary>
     <div class="accordion-content">
         <?= $this->render('subtask/table', array(
             'subtasks' => $subtasks,
@@ -9,4 +7,4 @@
             'editable' => $editable
         )) ?>
     </div>
-</section>
+</details>
diff --git a/app/Template/task/description.php b/app/Template/task/description.php
index f8e313dd..241b837a 100644
--- a/app/Template/task/description.php
+++ b/app/Template/task/description.php
@@ -1,10 +1,8 @@
-<section class="accordion-section <?= empty($task['description']) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Description') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($task['description']) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Description') ?></summary>
     <div class="accordion-content">
         <article class="markdown">
             <?= $this->text->markdown($task['description'], isset($is_public) && $is_public) ?>
         </article>
     </div>
-</section>
+</details>
diff --git a/app/Template/task_comments/show.php b/app/Template/task_comments/show.php
index 47bb5ff2..9010437c 100644
--- a/app/Template/task_comments/show.php
+++ b/app/Template/task_comments/show.php
@@ -1,7 +1,5 @@
-<section class="accordion-section <?= empty($comments) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Comments') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($comments) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Comments') ?></summary>
     <div class="accordion-content comments" id="comments">
         <?php if (!isset($is_public) || !$is_public): ?>
             <div class="comment-sorting">
@@ -35,4 +33,4 @@
             )) ?>
         <?php endif ?>
     </div>
-</section>
+</details>
diff --git a/app/Template/task_external_link/show.php b/app/Template/task_external_link/show.php
index 8ed11803..1bb89970 100644
--- a/app/Template/task_external_link/show.php
+++ b/app/Template/task_external_link/show.php
@@ -1,7 +1,5 @@
-<section class="accordion-section <?= empty($links) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('External links') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($links) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('External links') ?></summary>
     <div class="accordion-content">
         <?= $this->render('task_external_link/table', array(
             'links' => $links,
@@ -9,4 +7,4 @@
             'project' => $project,
         )) ?>
     </div>
-</section>
+</details>
diff --git a/app/Template/task_file/show.php b/app/Template/task_file/show.php
index df69605d..e9723f0d 100644
--- a/app/Template/task_file/show.php
+++ b/app/Template/task_file/show.php
@@ -1,9 +1,7 @@
-<section class="accordion-section <?= empty($files) && empty($images) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Attachments') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($files) && empty($images) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Attachments') ?></summary>
     <div class="accordion-content">
         <?= $this->render('task_file/images', array('task' => $task, 'images' => $images)) ?>
         <?= $this->render('task_file/files', array('task' => $task, 'files' => $files)) ?>
     </div>
-</section>
+</details>
diff --git a/app/Template/task_internal_link/show.php b/app/Template/task_internal_link/show.php
index cc05429c..e5c03898 100644
--- a/app/Template/task_internal_link/show.php
+++ b/app/Template/task_internal_link/show.php
@@ -1,7 +1,5 @@
-<section class="accordion-section <?= empty($links) ? 'accordion-collapsed' : '' ?>">
-    <div class="accordion-title">
-        <h3><a href="#" class="fa accordion-toggle"></a> <?= t('Internal links') ?></h3>
-    </div>
+<details class="accordion-section" <?= empty($links) ? '' : 'open' ?>>
+    <summary class="accordion-title"><?= t('Internal links') ?></summary>
     <div class="accordion-content">
         <?= $this->render('task_internal_link/table', array(
             'links' => $links,
@@ -11,4 +9,4 @@
             'is_public' => $is_public,
         )) ?>
     </div>
-</section>
+</details>