From d83756b4d990837b622d4dd28d674acd207a4ffa Mon Sep 17 00:00:00 2001
From: Jens Langhammer <jens.langhammer@beryju.org>
Date: Sun, 4 Apr 2021 16:20:49 +0200
Subject: [PATCH] web/admin: use expandable for Task list

Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>
---
 .../pages/system-tasks/SystemTaskListPage.ts  | 37 ++++++++++++++++---
 1 file changed, 32 insertions(+), 5 deletions(-)

diff --git a/web/src/pages/system-tasks/SystemTaskListPage.ts b/web/src/pages/system-tasks/SystemTaskListPage.ts
index 1b818d1ad..4c875cd8e 100644
--- a/web/src/pages/system-tasks/SystemTaskListPage.ts
+++ b/web/src/pages/system-tasks/SystemTaskListPage.ts
@@ -1,7 +1,8 @@
 import { t } from "@lingui/macro";
-import { customElement, html, property, TemplateResult } from "lit-element";
+import { CSSResult, customElement, html, property, TemplateResult } from "lit-element";
 import { AKResponse } from "../../api/Client";
 import { TablePage } from "../../elements/table/TablePage";
+import PFDescriptionList from "@patternfly/patternfly/components/DescriptionList/description-list.css";
 
 import "../../elements/buttons/SpinnerButton";
 import "../../elements/buttons/ActionButton";
@@ -24,9 +25,15 @@ export class SystemTaskListPage extends TablePage<Task> {
         return "pf-icon pf-icon-automation";
     }
 
+    expandable = true;
+
     @property()
     order = "slug";
 
+    static get styles(): CSSResult[] {
+        return super.styles.concat(PFDescriptionList);
+    }
+
     apiEndpoint(page: number): Promise<AKResponse<Task>> {
         return new AdminApi(DEFAULT_CONFIG).adminSystemTasksList().then((tasks) => {
             return {
@@ -48,7 +55,6 @@ export class SystemTaskListPage extends TablePage<Task> {
             new TableColumn("Description"),
             new TableColumn("Last run"),
             new TableColumn("Status"),
-            new TableColumn("Messages"),
             new TableColumn(""),
         ];
     }
@@ -66,15 +72,36 @@ export class SystemTaskListPage extends TablePage<Task> {
         }
     }
 
+    renderExpanded(item: Task): TemplateResult {
+        return html`
+        <td role="cell" colspan="3">
+            <div class="pf-c-table__expandable-row-content">
+                <dl class="pf-c-description-list pf-m-horizontal">
+                    <div class="pf-c-description-list__group">
+                        <dt class="pf-c-description-list__term">
+                            <span class="pf-c-description-list__text">${t`Messages`}</span>
+                        </dt>
+                        <dd class="pf-c-description-list__description">
+                            <div class="pf-c-description-list__text">
+                                ${item.messages.map(m => {
+                                    return html`<li>${m}</li>`;
+                                })}
+                            </div>
+                        </dd>
+                    </div>
+                </dl>
+            </div>
+        </td>
+        <td></td>
+        <td></td>`;
+    }
+
     row(item: Task): TemplateResult[] {
         return [
             html`${item.taskName}`,
             html`${item.taskDescription}`,
             html`${item.taskFinishTimestamp.toLocaleString()}`,
             this.taskStatus(item),
-            html`${item.messages.map(m => {
-                return html`<li>${m}</li>`;
-            })}`,
             html`<ak-action-button
                 .apiRequest=${() => {
                     return new AdminApi(DEFAULT_CONFIG).adminSystemTasksRetry({