fix unused

home-assistant · Jun 26, 2024 · 8336f9d · 8336f9d
1 parent 24400a1
commit 8336f9d
Show file tree

Hide file tree

Showing 3 changed files with 69 additions and 83 deletions.
diff --git a/src/panels/config/blueprint/ha-blueprint-overview.ts b/src/panels/config/blueprint/ha-blueprint-overview.ts
@@ -154,8 +154,6 @@ class HaBlueprintOverview extends LitElement {
 
   private _columns = memoizeOne(
     (
-      narrow,
-      _language,
       localize: LocalizeFunc
     ): DataTableColumnContainer<BlueprintMetaDataPath> => ({
       name: {
@@ -275,11 +273,7 @@ class HaBlueprintOverview extends LitElement {
         back-path="/config"
         .route=${this.route}
         .tabs=${configSections.automations}
-        .columns=${this._columns(
-          this.narrow,
-          this.hass.language,
-          this.hass.localize
-        )}
+        .columns=${this._columns(this.hass.localize)}
         .data=${this._processedBlueprints(this.blueprints, this.hass.localize)}
         id="fullpath"
         .noDataText=${this.hass.localize(

diff --git a/src/panels/config/scene/ha-scene-dashboard.ts b/src/panels/config/scene/ha-scene-dashboard.ts
@@ -225,7 +225,7 @@ class HaSceneDashboard extends SubscribeMixin(LitElement) {
   );
 
   private _columns = memoizeOne(
-    (narrow, localize: LocalizeFunc): DataTableColumnContainer => {
+    (localize: LocalizeFunc): DataTableColumnContainer => {
       const columns: DataTableColumnContainer<SceneItem> = {
         icon: {
           title: "",
@@ -539,7 +539,7 @@ class HaSceneDashboard extends SubscribeMixin(LitElement) {
                 Array.isArray(val) ? val.length : val
               )
         ).length}
-        .columns=${this._columns(this.narrow, this.hass.localize)}
+        .columns=${this._columns(this.hass.localize)}
         id="entity_id"
         .initialGroupColumn=${this._activeGrouping || "category"}
         .initialCollapsedGroups=${this._activeCollapsed}

diff --git a/src/panels/config/tags/ha-config-tags.ts b/src/panels/config/tags/ha-config-tags.ts
@@ -66,86 +66,82 @@ export class HaConfigTags extends SubscribeMixin(LitElement) {
   })
   private _filter = "";
 
-  private _columns = memoizeOne(
-    (narrow: boolean, _language, localize: LocalizeFunc) => {
-      const columns: DataTableColumnContainer<TagRowData> = {
-        icon: {
-          title: "",
-          moveable: false,
-          showNarrow: true,
-          label: localize("ui.panel.config.tag.headers.icon"),
-          type: "icon",
-          template: (tag) => html`<tag-image .tag=${tag}></tag-image>`,
-        },
-        display_name: {
-          title: localize("ui.panel.config.tag.headers.name"),
-          main: true,
-          sortable: true,
-          filterable: true,
-          grows: true,
-        },
-        last_scanned_datetime: {
-          title: localize("ui.panel.config.tag.headers.last_scanned"),
-          sortable: true,
-          direction: "desc",
-          width: "20%",
-          template: (tag) => html`
-            ${tag.last_scanned_datetime
-              ? html`<ha-relative-time
-                  .hass=${this.hass}
-                  .datetime=${tag.last_scanned_datetime}
-                  capitalize
-                ></ha-relative-time>`
-              : this.hass.localize("ui.panel.config.tag.never_scanned")}
-          `,
-        },
-      };
-      if (this._canWriteTags) {
-        columns.write = {
-          title: "",
-          label: localize("ui.panel.config.tag.headers.write"),
-          type: "icon-button",
-          showNarrow: true,
-          template: (tag) =>
-            html`<ha-icon-button
-              .tag=${tag}
-              @click=${this._handleWriteClick}
-              .label=${this.hass.localize("ui.panel.config.tag.write")}
-              .path=${mdiContentDuplicate}
-            ></ha-icon-button>`,
-        };
-      }
-      columns.automation = {
+  private _columns = memoizeOne((localize: LocalizeFunc) => {
+    const columns: DataTableColumnContainer<TagRowData> = {
+      icon: {
         title: "",
-        type: "icon-button",
+        moveable: false,
         showNarrow: true,
-        template: (tag) =>
-          html`<ha-icon-button
-            .tag=${tag}
-            @click=${this._handleAutomationClick}
-            .label=${this.hass.localize(
-              "ui.panel.config.tag.create_automation"
-            )}
-            .path=${mdiRobot}
-          ></ha-icon-button>`,
-      };
-      columns.edit = {
+        label: localize("ui.panel.config.tag.headers.icon"),
+        type: "icon",
+        template: (tag) => html`<tag-image .tag=${tag}></tag-image>`,
+      },
+      display_name: {
+        title: localize("ui.panel.config.tag.headers.name"),
+        main: true,
+        sortable: true,
+        filterable: true,
+        grows: true,
+      },
+      last_scanned_datetime: {
+        title: localize("ui.panel.config.tag.headers.last_scanned"),
+        sortable: true,
+        direction: "desc",
+        width: "20%",
+        template: (tag) => html`
+          ${tag.last_scanned_datetime
+            ? html`<ha-relative-time
+                .hass=${this.hass}
+                .datetime=${tag.last_scanned_datetime}
+                capitalize
+              ></ha-relative-time>`
+            : this.hass.localize("ui.panel.config.tag.never_scanned")}
+        `,
+      },
+    };
+    if (this._canWriteTags) {
+      columns.write = {
         title: "",
+        label: localize("ui.panel.config.tag.headers.write"),
         type: "icon-button",
         showNarrow: true,
-        hideable: false,
-        moveable: false,
         template: (tag) =>
           html`<ha-icon-button
             .tag=${tag}
-            @click=${this._handleEditClick}
-            .label=${this.hass.localize("ui.panel.config.tag.edit")}
-            .path=${mdiCog}
+            @click=${this._handleWriteClick}
+            .label=${this.hass.localize("ui.panel.config.tag.write")}
+            .path=${mdiContentDuplicate}
           ></ha-icon-button>`,
       };
-      return columns;
     }
-  );
+    columns.automation = {
+      title: "",
+      type: "icon-button",
+      showNarrow: true,
+      template: (tag) =>
+        html`<ha-icon-button
+          .tag=${tag}
+          @click=${this._handleAutomationClick}
+          .label=${this.hass.localize("ui.panel.config.tag.create_automation")}
+          .path=${mdiRobot}
+        ></ha-icon-button>`,
+    };
+    columns.edit = {
+      title: "",
+      type: "icon-button",
+      showNarrow: true,
+      hideable: false,
+      moveable: false,
+      template: (tag) =>
+        html`<ha-icon-button
+          .tag=${tag}
+          @click=${this._handleEditClick}
+          .label=${this.hass.localize("ui.panel.config.tag.edit")}
+          .path=${mdiCog}
+        ></ha-icon-button>`,
+    };
+    return columns;
+  });
 
   private _data = memoizeOne((tags: Tag[]): TagRowData[] =>
     tags.map((tag) => ({
@@ -184,11 +180,7 @@ export class HaConfigTags extends SubscribeMixin(LitElement) {
         back-path="/config"
         .route=${this.route}
         .tabs=${configSections.tags}
-        .columns=${this._columns(
-          this.narrow,
-          this.hass.language,
-          this.hass.localize
-        )}
+        .columns=${this._columns(this.hass.localize)}
         .data=${this._data(this._tags)}
         .noDataText=${this.hass.localize("ui.panel.config.tag.no_tags")}
         .filter=${this._filter}