Anpassung des Theme Colors (+andere)

[philipp123]

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

Das ist ja kein Thema dieses Themes sondern ein allgemeines?

[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]

@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.

[electron]

So, die Änderung ist eingereicht. Ich hoffe es funktiniert. Mein Testsystem hat keine historischen Daten, daher konnte ich es nur begrenzt testen.

Danke, passt. Hat ja keine Eile. Hatte nur Angst, dass es untergeht

[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.

@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.

[LutzB]

Da sollte anstatt "de-DE" einfach "undefined" rein. Dann nimmt die Funktion die Einstellungen des Browsers.

[electron]

Danke, das hat funktioniert!
PR ist eingereicht.

[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.

[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.

[hhoefling]

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