Remove narrow from strategies (#18201)

This commit is contained in:
Bram Kragten 2023-10-16 06:20:36 +02:00 committed by GitHub
parent 02d9786f8c
commit 4b885cbd93
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
9 changed files with 20 additions and 54 deletions

View File

@ -323,8 +323,7 @@ export class HcMain extends HassElement {
{ {
type: DEFAULT_STRATEGY, type: DEFAULT_STRATEGY,
}, },
this.hass!, this.hass!
{ narrow: false }
) )
); );
} }

View File

@ -174,9 +174,7 @@ export class HuiSaveConfig extends LitElement implements HassDialog {
await lovelace.saveConfig( await lovelace.saveConfig(
this._emptyConfig this._emptyConfig
? EMPTY_CONFIG ? EMPTY_CONFIG
: await expandLovelaceConfigStrategies(lovelace.config, this.hass, { : await expandLovelaceConfigStrategies(lovelace.config, this.hass)
narrow: this._params.narrow,
})
); );
lovelace.setEditMode(true); lovelace.setEditMode(true);
this._saving = false; this._saving = false;

View File

@ -167,8 +167,7 @@ export class LovelacePanel extends LitElement {
{ {
type: DEFAULT_STRATEGY, type: DEFAULT_STRATEGY,
}, },
this.hass!, this.hass!
{ narrow: this.narrow }
); );
this._setLovelaceConfig(conf, undefined, "generated"); this._setLovelaceConfig(conf, undefined, "generated");
this._panelState = "loaded"; this._panelState = "loaded";
@ -258,8 +257,7 @@ export class LovelacePanel extends LitElement {
if (rawConf.strategy) { if (rawConf.strategy) {
conf = await generateLovelaceDashboardStrategy( conf = await generateLovelaceDashboardStrategy(
rawConf.strategy, rawConf.strategy,
this.hass!, this.hass!
{ narrow: this.narrow }
); );
} else { } else {
conf = rawConf; conf = rawConf;
@ -276,8 +274,7 @@ export class LovelacePanel extends LitElement {
{ {
type: DEFAULT_STRATEGY, type: DEFAULT_STRATEGY,
}, },
this.hass!, this.hass!
{ narrow: this.narrow }
); );
confMode = "generated"; confMode = "generated";
} finally { } finally {
@ -365,8 +362,7 @@ export class LovelacePanel extends LitElement {
if (newConfig.strategy) { if (newConfig.strategy) {
conf = await generateLovelaceDashboardStrategy( conf = await generateLovelaceDashboardStrategy(
newConfig.strategy, newConfig.strategy,
this.hass!, this.hass!
{ narrow: this.narrow }
); );
} else { } else {
conf = newConfig; conf = newConfig;
@ -404,8 +400,7 @@ export class LovelacePanel extends LitElement {
{ {
type: DEFAULT_STRATEGY, type: DEFAULT_STRATEGY,
}, },
this.hass!, this.hass!
{ narrow: this.narrow }
); );
this._updateLovelace({ this._updateLovelace({
config: generatedConf, config: generatedConf,

View File

@ -8,7 +8,6 @@ import { isLegacyStrategy } from "./legacy-strategy";
import { import {
LovelaceDashboardStrategy, LovelaceDashboardStrategy,
LovelaceStrategy, LovelaceStrategy,
LovelaceStrategyParams,
LovelaceViewStrategy, LovelaceViewStrategy,
} from "./types"; } from "./types";
@ -77,8 +76,7 @@ const generateStrategy = async <T extends LovelaceStrategyConfigType>(
configType: T, configType: T,
renderError: (err: string | Error) => StrategyConfig<T>, renderError: (err: string | Error) => StrategyConfig<T>,
strategyConfig: LovelaceStrategyConfig, strategyConfig: LovelaceStrategyConfig,
hass: HomeAssistant, hass: HomeAssistant
params: LovelaceStrategyParams
): Promise<StrategyConfig<T>> => { ): Promise<StrategyConfig<T>> => {
const strategyType = strategyConfig.type; const strategyType = strategyConfig.type;
if (!strategyType) { if (!strategyType) {
@ -95,7 +93,6 @@ const generateStrategy = async <T extends LovelaceStrategyConfigType>(
return (await strategy.generateDashboard({ return (await strategy.generateDashboard({
config: { strategy: strategyConfig, views: [] }, config: { strategy: strategyConfig, views: [] },
hass, hass,
narrow: params.narrow,
})) as StrategyConfig<T>; })) as StrategyConfig<T>;
} }
if (configType === "view" && "generateView" in strategy) { if (configType === "view" && "generateView" in strategy) {
@ -103,7 +100,6 @@ const generateStrategy = async <T extends LovelaceStrategyConfigType>(
config: { views: [] }, config: { views: [] },
view: { strategy: strategyConfig }, view: { strategy: strategyConfig },
hass, hass,
narrow: params.narrow,
})) as StrategyConfig<T>; })) as StrategyConfig<T>;
} }
} }
@ -115,7 +111,7 @@ const generateStrategy = async <T extends LovelaceStrategyConfigType>(
delete config.options; delete config.options;
return await strategy.generate(config, hass, params); return await strategy.generate(config, hass);
} catch (err: any) { } catch (err: any) {
if (err.message !== "timeout") { if (err.message !== "timeout") {
// eslint-disable-next-line // eslint-disable-next-line
@ -128,8 +124,7 @@ const generateStrategy = async <T extends LovelaceStrategyConfigType>(
export const generateLovelaceDashboardStrategy = async ( export const generateLovelaceDashboardStrategy = async (
strategyConfig: LovelaceStrategyConfig, strategyConfig: LovelaceStrategyConfig,
hass: HomeAssistant, hass: HomeAssistant
params: LovelaceStrategyParams
): Promise<LovelaceConfig> => ): Promise<LovelaceConfig> =>
generateStrategy( generateStrategy(
"dashboard", "dashboard",
@ -147,14 +142,12 @@ export const generateLovelaceDashboardStrategy = async (
], ],
}), }),
strategyConfig, strategyConfig,
hass, hass
params
); );
export const generateLovelaceViewStrategy = async ( export const generateLovelaceViewStrategy = async (
strategyConfig: LovelaceStrategyConfig, strategyConfig: LovelaceStrategyConfig,
hass: HomeAssistant, hass: HomeAssistant
params: LovelaceStrategyParams
): Promise<LovelaceViewConfig> => ): Promise<LovelaceViewConfig> =>
generateStrategy( generateStrategy(
"view", "view",
@ -167,8 +160,7 @@ export const generateLovelaceViewStrategy = async (
], ],
}), }),
strategyConfig, strategyConfig,
hass, hass
params
); );
/** /**
@ -176,18 +168,15 @@ export const generateLovelaceViewStrategy = async (
*/ */
export const expandLovelaceConfigStrategies = async ( export const expandLovelaceConfigStrategies = async (
config: LovelaceConfig, config: LovelaceConfig,
hass: HomeAssistant, hass: HomeAssistant
params: LovelaceStrategyParams
): Promise<LovelaceConfig> => { ): Promise<LovelaceConfig> => {
const newConfig = config.strategy const newConfig = config.strategy
? await generateLovelaceDashboardStrategy(config.strategy, hass, params) ? await generateLovelaceDashboardStrategy(config.strategy, hass)
: { ...config }; : { ...config };
newConfig.views = await Promise.all( newConfig.views = await Promise.all(
newConfig.views.map((view) => newConfig.views.map((view) =>
view.strategy view.strategy ? generateLovelaceViewStrategy(view.strategy, hass) : view
? generateLovelaceViewStrategy(view.strategy, hass, params)
: view
) )
); );

View File

@ -10,7 +10,6 @@ export interface LovelaceDashboardStrategy {
generateDashboard(info: { generateDashboard(info: {
config?: LovelaceConfig; config?: LovelaceConfig;
hass: HomeAssistant; hass: HomeAssistant;
narrow: boolean | undefined;
}): Promise<LovelaceConfig>; }): Promise<LovelaceConfig>;
} }
@ -19,6 +18,5 @@ export interface LovelaceViewStrategy {
view: LovelaceViewConfig; view: LovelaceViewConfig;
config: LovelaceConfig; config: LovelaceConfig;
hass: HomeAssistant; hass: HomeAssistant;
narrow: boolean | undefined;
}): Promise<LovelaceViewConfig>; }): Promise<LovelaceViewConfig>;
} }

View File

@ -2,14 +2,12 @@ import { ReactiveElement } from "lit";
import { customElement } from "lit/decorators"; import { customElement } from "lit/decorators";
import { LovelaceConfig, LovelaceStrategyConfig } from "../../../data/lovelace"; import { LovelaceConfig, LovelaceStrategyConfig } from "../../../data/lovelace";
import { HomeAssistant } from "../../../types"; import { HomeAssistant } from "../../../types";
import { LovelaceStrategyParams } from "./types";
@customElement("original-states-dashboard-strategy") @customElement("original-states-dashboard-strategy")
export class OriginalStatesDashboardStrategy extends ReactiveElement { export class OriginalStatesDashboardStrategy extends ReactiveElement {
static async generate( static async generate(
_config: LovelaceStrategyConfig, _config: LovelaceStrategyConfig,
hass: HomeAssistant, hass: HomeAssistant
_params?: LovelaceStrategyParams
): Promise<LovelaceConfig> { ): Promise<LovelaceConfig> {
return { return {
title: hass.config.location_name, title: hass.config.location_name,

View File

@ -9,14 +9,12 @@ import {
} from "../../../data/lovelace"; } from "../../../data/lovelace";
import { HomeAssistant } from "../../../types"; import { HomeAssistant } from "../../../types";
import { generateDefaultViewConfig } from "../common/generate-lovelace-config"; import { generateDefaultViewConfig } from "../common/generate-lovelace-config";
import { LovelaceStrategyParams } from "./types";
@customElement("original-states-view-strategy") @customElement("original-states-view-strategy")
export class OriginalStatesViewStrategy extends ReactiveElement { export class OriginalStatesViewStrategy extends ReactiveElement {
static async generate( static async generate(
_config: LovelaceStrategyConfig, _config: LovelaceStrategyConfig,
hass: HomeAssistant, hass: HomeAssistant
_params?: LovelaceStrategyParams
): Promise<LovelaceViewConfig> { ): Promise<LovelaceViewConfig> {
if (hass.config.state === STATE_NOT_RUNNING) { if (hass.config.state === STATE_NOT_RUNNING) {
return { return {

View File

@ -5,16 +5,8 @@ import {
} from "../../../data/lovelace"; } from "../../../data/lovelace";
import { HomeAssistant } from "../../../types"; import { HomeAssistant } from "../../../types";
export type LovelaceStrategyParams = {
narrow?: boolean;
};
export type LovelaceStrategy<T = any> = { export type LovelaceStrategy<T = any> = {
generate( generate(config: LovelaceStrategyConfig, hass: HomeAssistant): Promise<T>;
config: LovelaceStrategyConfig,
hass: HomeAssistant,
params?: LovelaceStrategyParams
): Promise<T>;
}; };
export interface LovelaceDashboardStrategy export interface LovelaceDashboardStrategy

View File

@ -192,8 +192,7 @@ export class HUIView extends ReactiveElement {
isStrategy = true; isStrategy = true;
viewConfig = await generateLovelaceViewStrategy( viewConfig = await generateLovelaceViewStrategy(
viewConfig.strategy, viewConfig.strategy,
this.hass!, this.hass!
{ narrow: this.narrow }
); );
} }