Add a github option when copying from system health (#7663)

Co-authored-by: Bram Kragten <mail@bramkragten.nl>

src/panels/config/info/system-health-card.ts

@@ -1,7 +1,9 @@
 import "@material/mwc-button/mwc-button";
 import "@material/mwc-icon-button";
-import "../../../components/ha-circular-progress";
+import { ActionDetail } from "@material/mwc-list/mwc-list-foundation";
+import "@material/mwc-list/mwc-list-item";
 import { mdiContentCopy } from "@mdi/js";
+import "@polymer/paper-tooltip/paper-tooltip";
 import {
   css,
   CSSResult,
@@ -9,21 +11,22 @@ import {
   internalProperty,
   LitElement,
   property,
-  query,
   TemplateResult,
 } from "lit-element";
+import { formatDateTime } from "../../../common/datetime/format_date_time";
+import { copyToClipboard } from "../../../common/util/copy-clipboard";
+import "../../../components/ha-button-menu";
 import "../../../components/ha-card";
-import "@polymer/paper-tooltip/paper-tooltip";
-import type { PaperTooltipElement } from "@polymer/paper-tooltip/paper-tooltip";
+import "../../../components/ha-circular-progress";
+import "../../../components/ha-svg-icon";
 import { domainToName } from "../../../data/integration";
 import {
   subscribeSystemHealthInfo,
-  SystemHealthInfo,
   SystemCheckValueObject,
+  SystemHealthInfo,
 } from "../../../data/system_health";
 import { HomeAssistant } from "../../../types";
-import { formatDateTime } from "../../../common/datetime/format_date_time";
-import { copyToClipboard } from "../../../common/util/copy-clipboard";
+import { showToast } from "../../../util/toast";
 
 const sortKeys = (a: string, b: string) => {
   if (a === "homeassistant") {
@@ -46,8 +49,6 @@ class SystemHealthCard extends LitElement {
 
   @internalProperty() private _info?: SystemHealthInfo;
 
-  @query("paper-tooltip", true) private _toolTip?: PaperTooltipElement;
-
   protected render(): TemplateResult {
     if (!this.hass) {
       return html``;
@@ -152,18 +153,21 @@ class SystemHealthCard extends LitElement {
           <div class="card-header-text">
             ${domainToName(this.hass.localize, "system_health")}
           </div>
-          <mwc-icon-button id="copy" @click=${this._copyInfo}>
-            <ha-svg-icon .path=${mdiContentCopy}></ha-svg-icon>
-          </mwc-icon-button>
-          <paper-tooltip
-            manual-mode
-            for="copy"
-            position="left"
-            animation-delay="0"
-            offset="4"
+          <ha-button-menu
+            corner="BOTTOM_START"
+            slot="toolbar-icon"
+            @action=${this._copyInfo}
           >
-            ${this.hass.localize("ui.common.copied")}
-          </paper-tooltip>
+            <mwc-icon-button slot="trigger" alt="menu">
+              <ha-svg-icon .path=${mdiContentCopy}></ha-svg-icon>
+            </mwc-icon-button>
+            <mwc-list-item>
+              ${this.hass.localize("ui.panel.config.info.copy_raw")}
+            </mwc-list-item>
+            <mwc-list-item>
+              ${this.hass.localize("ui.panel.config.info.copy_github")}
+            </mwc-list-item>
+          </ha-button-menu>
         </h1>
         <div class="card-content">${sections}</div>
       </ha-card>
@@ -193,13 +197,24 @@ class SystemHealthCard extends LitElement {
     });
   }
 
-  private _copyInfo(): void {
+  private _copyInfo(ev: CustomEvent<ActionDetail>): void {
+    const github = ev.detail.index === 1;
     let haContent: string | undefined;
     const domainParts: string[] = [];
 
     for (const domain of Object.keys(this._info!).sort(sortKeys)) {
       const domainInfo = this._info![domain];
-      const parts = [`${domainToName(this.hass.localize, domain)}\n`];
+      let first = true;
+      const parts = [
+        `${
+          github && domain !== "homeassistant"
+            ? `<details><summary>${domainToName(
+                this.hass.localize,
+                domain
+              )}</summary>\n`
+            : ""
+        }`,
+      ];
 
       for (const key of Object.keys(domainInfo.info)) {
         let value: unknown;
@@ -217,23 +232,31 @@ class SystemHealthCard extends LitElement {
         } else {
           value = domainInfo.info[key];
         }
-
-        parts.push(`${key}: ${value}`);
+        if (github && first) {
+          parts.push(`${key} | ${value}\n-- | --`);
+          first = false;
+        } else {
+          parts.push(`${key}${github ? " | " : ": "}${value}`);
+        }
       }
 
       if (domain === "homeassistant") {
         haContent = parts.join("\n");
       } else {
         domainParts.push(parts.join("\n"));
+        if (github && domain !== "homeassistant") {
+          domainParts.push("</details>");
+        }
       }
     }
 
     copyToClipboard(
-      `System Health\n\n${haContent}\n\n${domainParts.join("\n\n")}`
+      `${github ? "## " : ""}System Health\n${haContent}\n\n${domainParts.join(
+        "\n\n"
+      )}`
     );
 
-    this._toolTip!.show();
-    setTimeout(() => this._toolTip?.hide(), 3000);
+    showToast(this, { message: this.hass.localize("ui.common.copied") });
   }
 
   static get styles(): CSSResult {

src/translations/en.json

@@ -899,6 +899,8 @@
         },
         "info": {
           "caption": "Info",
+          "copy_raw": "Raw Text",
+          "copy_github": "For GitHub",
           "description": "View info about your Home Assistant installation",
           "home_assistant_logo": "Home Assistant logo",
           "path_configuration": "Path to configuration.yaml: {path}",