From 1f2f5d2c102b65abf44eeaa9b7864566a409ff80 Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Sun, 23 Aug 2015 11:09:14 -0400
Subject: Improve file attachments tooltip on the board

---
 app/Controller/Board.php             |  3 +--
 app/Template/board/tooltip_files.php | 49 +++++++++++++-----------------------
 2 files changed, 19 insertions(+), 33 deletions(-)

(limited to 'app')

diff --git a/app/Controller/Board.php b/app/Controller/Board.php
index 976e44fc..179c6b3c 100644
--- a/app/Controller/Board.php
+++ b/app/Controller/Board.php
@@ -179,8 +179,7 @@ class Board extends Base
         $task = $this->getTask();
 
         $this->response->html($this->template->render('board/tooltip_files', array(
-            'files' => $this->file->getAllDocuments($task['id']),
-            'images' => $this->file->getAllImages($task['id']),
+            'files' => $this->file->getAll($task['id']),
             'task' => $task,
         )));
     }
diff --git a/app/Template/board/tooltip_files.php b/app/Template/board/tooltip_files.php
index 70e3d371..96428d3b 100644
--- a/app/Template/board/tooltip_files.php
+++ b/app/Template/board/tooltip_files.php
@@ -1,31 +1,18 @@
-<section>
-    <table>
-        <?php if (! empty($images)): ?>
-            <?php foreach ($images as $file): ?>
-                <tr>
-                    <td class="column-70">
-                        <i class="fa fa-file-image-o fa-fw"></i>
-                        <?= $this->e($file['name']) ?>
-                    </td>
-                    <td>
-                        <i class="fa fa-download"></i> <?= $this->url->link(t('download'), 'file', 'download', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'file_id' => $file['id'])) ?>
-                        <i class="fa fa-eye"></i> <?= $this->url->link(t('open file'), 'file', 'open', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'file_id' => $file['id']), false, 'popover') ?>
-                    </td>
-                </tr>
-            <?php endforeach ?>
-        <?php endif ?>
-        <?php if (! empty($files)): ?>
-            <?php foreach ($files as $file): ?>
-                <tr>
-                    <td>
-                        <i class="fa <?= $this->file->icon($file['name']) ?> fa-fw"></i>
-                        <?= $this->e($file['name']) ?>
-                    </td>
-                    <td>
-                        <i class="fa fa-download"></i> <?= $this->url->link(t('download'), 'file', 'download', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'file_id' => $file['id'])) ?>
-                    </td>
-                </tr>
-            <?php endforeach ?>
-        <?php endif ?>
-    </table>
-</section>
+<table class="table-small">
+    <?php foreach($files as $file): ?>
+    <tr>
+        <th>
+            <i class="fa <?= $this->file->icon($file['name']) ?> fa-fw"></i>
+            <?= $this->e($file['name']) ?>
+        </th>
+    </tr>
+    <tr>
+        <td>
+            <i class="fa fa-download fa-fw"></i><?= $this->url->link(t('download'), 'file', 'download', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'file_id' => $file['id'])) ?>
+            <?php if ($file['is_image'] == 1): ?>
+                &nbsp;<i class="fa fa-eye"></i> <?= $this->url->link(t('open file'), 'file', 'open', array('task_id' => $task['id'], 'project_id' => $task['project_id'], 'file_id' => $file['id']), false, 'popover') ?>
+            <?php endif ?>
+        </td>
+    </tr>
+    <?php endforeach ?>
+</table>
\ No newline at end of file
-- 
cgit v1.2.3