Anpassung des Theme Colors (+andere)

Auflistung von gewünschten Features, Ausschreibung zur Umsetzung
philipp123
Beiträge: 1032
Registriert: Mi Jul 21, 2021 3:00 pm

Re: Anpassung des Theme Colors (+andere)

Beitrag von philipp123 »

thesorrow101 hat geschrieben: Mi Mär 02, 2022 7:14 am Anfrage:

Ist es möglich nach Auswahl des Themes wieder auf die Hauptseite weitergeleitet zu werden oder eine Meldung zu bekommen?

Derzeit gibt es nach Auswahl des Themes kein wirkliches Feedback. Wenn man zur Startseite weitergeleitet wird, dann weiß man wenigstens dass die Eingabe angekommen ist :D
Das ist ja kein Thema dieses Themes sondern ein allgemeines?
LP1: openWB series2 custom mit Phasenumschaltung
LP2: go-e V2
Kostal Plenticore Plus
e-up BJ 2021, SOC mit OVMS
EQB 250 BJ 2023, SOC mit Mercedes ME über Home Assistant
EVU mit Tasmato-Lesekopf auf SmartMeter
9 x Smarthome mit Shellys
thesorrow101
Beiträge: 685
Registriert: Mo Okt 12, 2020 7:41 am

Re: Anpassung des Theme Colors (+andere)

Beitrag von thesorrow101 »

Korrekt, nachdem der Thread aber Theme Colors (+andere) heißt hoffte ich mich hier richtig verortet :)

Aber ich kann es auch verschieben, wenn ihr denkt er wäre woanders besser?
hhoefling
Beiträge: 627
Registriert: So Jul 25, 2021 2:32 pm

Re: Anpassung des Theme Colors (+andere)

Beitrag von hhoefling »

@electron

Ich habe bei mir diese beiden Functionen in powerdata.js ausgetauscht.

Code: Alles auswählen

function formatWatt(watt) {
	if (watt >= 1000) {
		switch (wbdata.decimalPlaces) {
			case 0:	return Math.round(watt / 1000).toLocaleString('de-DE')  + " kW";
					break;
			case 1: return (Math.round(watt / 100) / 10).toLocaleString('de-DE') + " kW";
					break;
			case 2: return (Math.round(watt / 10) / 100).toLocaleString('de-DE') + " kW";
					break;
			case 3: return (Math.round(watt) /* / 1000*/).toLocaleString('de-DE') + " W";
					break;
			default:return (Math.round(watt / 100) / 10 ).toLocaleString('de-DE') + " kW";
					break;
		}
	} else {
		return (Math.round(watt).toLocaleString('de-DE') + " W");
	}
}

function formatWattH(watt) {
	if (watt >= 1000) {
		switch (wbdata.decimalPlaces) {
			case 0:	return Math.round(watt / 1000).toLocaleString('de-DE') + ' kWh';
					break;
			case 1: return (Math.round(watt / 100) / 10).toLocaleString('de-DE') + ' kWh';
					break;
			case 2: return (Math.round(watt / 10) / 100).toLocaleString('de-DE') + ' kWh';
					break;
			case 3: return (Math.round(watt) /* / 1000*/).toLocaleString('de-DE')  + ' Wh';
					break;
			default:return (Math.round(watt / 100) / 10).toLocaleString('de-DE') + ' kWh';
					break;
		}
	} else {
		return (Math.round(watt).toLocaleString('de-DE') + " Wh");
	}
}
Damit bekomme ich nun die Zahlen etwas besser an unsere Verhältnisse angepasst.
aus 15 kW wird 15 kW
aus 15.4 kW wird 15,4 kW
aus 15.41 kW wird 15,41 kW
aus 15.414 kW wird 15.414 W *** tausender-Punkt!!! kein Komma.

Kannst du ja vieleicht demnächst mal mit übernehmen.
gruss
Heinz

6kWp PV+Akku von RCT Power,+Tibber,+Shelly's
Skoda Citigo e-iV, openWB series2 standart+
mit openWB_lite (auf Basis der openWB 1.9.244+ Okt.2021)
Github: https://github.com/hhoefling/openWB_lite
electron
Beiträge: 744
Registriert: So Aug 23, 2020 5:11 am

Re: Anpassung des Theme Colors (+andere)

Beitrag von electron »

So, die Änderung ist eingereicht. Ich hoffe es funktiniert. Mein Testsystem hat keine historischen Daten, daher konnte ich es nur begrenzt testen.
philipp123 hat geschrieben: Mi Mär 02, 2022 9:28 am
Danke, passt. Hat ja keine Eile. Hatte nur Angst, dass es untergeht :D
openWB Serie 2 11 kW
Solaredge WR
Tesla Model 3
electron
Beiträge: 744
Registriert: So Aug 23, 2020 5:11 am

Re: Anpassung des Theme Colors (+andere)

Beitrag von electron »

Das müsste aber dann automatisch die locale des Benutzer ermitteln und darüber steuern, wie das Dezimalzeichen angezeigt wird.
(Ich wohne z.B. in der Schweiz und da passt die aktuelle Darstellung :) - Mir war dadurch noch gar nicht aufgefallen, dass die Darstellung für Deutschland nicht korrekt ist)
Ich muss mal herausfinden, wie man die Browser-locale auslesen kann.
hhoefling hat geschrieben: Do Mär 03, 2022 6:02 pm @electron

Ich habe bei mir diese beiden Functionen in powerdata.js ausgetauscht.

Code: Alles auswählen

function formatWatt(watt) {
	if (watt >= 1000) {
		switch (wbdata.decimalPlaces) {
			case 0:	return Math.round(watt / 1000).toLocaleString('de-DE')  + " kW";
					break;
			case 1: return (Math.round(watt / 100) / 10).toLocaleString('de-DE') + " kW";
					break;
			case 2: return (Math.round(watt / 10) / 100).toLocaleString('de-DE') + " kW";
					break;
			case 3: return (Math.round(watt) /* / 1000*/).toLocaleString('de-DE') + " W";
					break;
			default:return (Math.round(watt / 100) / 10 ).toLocaleString('de-DE') + " kW";
					break;
		}
	} else {
		return (Math.round(watt).toLocaleString('de-DE') + " W");
	}
}

function formatWattH(watt) {
	if (watt >= 1000) {
		switch (wbdata.decimalPlaces) {
			case 0:	return Math.round(watt / 1000).toLocaleString('de-DE') + ' kWh';
					break;
			case 1: return (Math.round(watt / 100) / 10).toLocaleString('de-DE') + ' kWh';
					break;
			case 2: return (Math.round(watt / 10) / 100).toLocaleString('de-DE') + ' kWh';
					break;
			case 3: return (Math.round(watt) /* / 1000*/).toLocaleString('de-DE')  + ' Wh';
					break;
			default:return (Math.round(watt / 100) / 10).toLocaleString('de-DE') + ' kWh';
					break;
		}
	} else {
		return (Math.round(watt).toLocaleString('de-DE') + " Wh");
	}
}
Damit bekomme ich nun die Zahlen etwas besser an unsere Verhältnisse angepasst.
aus 15 kW wird 15 kW
aus 15.4 kW wird 15,4 kW
aus 15.41 kW wird 15,41 kW
aus 15.414 kW wird 15.414 W *** tausender-Punkt!!! kein Komma.

Kannst du ja vieleicht demnächst mal mit übernehmen.
openWB Serie 2 11 kW
Solaredge WR
Tesla Model 3
LutzB
Beiträge: 3479
Registriert: Di Feb 25, 2020 9:23 am

Re: Anpassung des Theme Colors (+andere)

Beitrag von LutzB »

Da sollte anstatt "de-DE" einfach "undefined" rein. Dann nimmt die Funktion die Einstellungen des Browsers.
electron
Beiträge: 744
Registriert: So Aug 23, 2020 5:11 am

Re: Anpassung des Theme Colors (+andere)

Beitrag von electron »

Danke, das hat funktioniert!
PR ist eingereicht.
openWB Serie 2 11 kW
Solaredge WR
Tesla Model 3
hhoefling
Beiträge: 627
Registriert: So Jul 25, 2021 2:32 pm

Re: Anpassung des Theme Colors (+andere)

Beitrag von hhoefling »

Danke.
funktioniert... fast.

bei dem Modus 3-Nachkommastellen würde ich doch
4.580 Wh / 25,4 km
statt
4,580 kWh / 25,4 km
bevorzugen.
gruss
Heinz

6kWp PV+Akku von RCT Power,+Tibber,+Shelly's
Skoda Citigo e-iV, openWB series2 standart+
mit openWB_lite (auf Basis der openWB 1.9.244+ Okt.2021)
Github: https://github.com/hhoefling/openWB_lite
philipp123
Beiträge: 1032
Registriert: Mi Jul 21, 2021 3:00 pm

Re: Anpassung des Theme Colors (+andere)

Beitrag von philipp123 »

Danke dir Claus. Super, dass jetzt die Geräte nicht mehr gruppiert werden. Gerade in der Monatssicht finde ich das einen tollen Mehrwert.

@hhoefling: Warum gerade bei 3 Nachkommastellen . statt ,? Gerade dort verwirrt es doch. Ich würde es durchgängig gleich lassen.
LP1: openWB series2 custom mit Phasenumschaltung
LP2: go-e V2
Kostal Plenticore Plus
e-up BJ 2021, SOC mit OVMS
EQB 250 BJ 2023, SOC mit Mercedes ME über Home Assistant
EVU mit Tasmato-Lesekopf auf SmartMeter
9 x Smarthome mit Shellys
hhoefling
Beiträge: 627
Registriert: So Jul 25, 2021 2:32 pm

Re: Anpassung des Theme Colors (+andere)

Beitrag von hhoefling »

Weil das dann kein Komma sondern ein "Tausender-Punkt" ist.
Spart uns das k und somit Platz.
gruss
Heinz

6kWp PV+Akku von RCT Power,+Tibber,+Shelly's
Skoda Citigo e-iV, openWB series2 standart+
mit openWB_lite (auf Basis der openWB 1.9.244+ Okt.2021)
Github: https://github.com/hhoefling/openWB_lite
Antworten