Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v3: Problem mit den Kommas in Secondary State hatten wir auch in v2.2.0 #791

Closed
mcuiobroker opened this issue Jun 12, 2021 · 43 comments
Closed
Assignees
Labels
bug 🐛 Something isn't working

Comments

@mcuiobroker
Copy link
Collaborator

Bei NullWerten oder Leer kein Komma davor und Einheit ausblenden

image
Grundsätzlich erscheint immer ein Komma vor den Werten.
image

@mcuiobroker mcuiobroker added the bug 🐛 Something isn't working label Jun 12, 2021
@Zefau
Copy link
Owner

Zefau commented Jun 13, 2021

Ist es mit der v3.0.0-alpha.8 besser?

@mcuiobroker
Copy link
Collaborator Author

Mit v3.0.0-alpha.8 ist es wesentlich besser geworden:
image
Nur noch bei Leerem Text und 0-Werten
image
image
image
Leerer text:
image
Und leeren Wert die Einheit verschwinden lassen, so wie in 2.2.0

@Zefau Zefau added this to the ROADMAP v3.0.0 milestone Jun 22, 2021
@Zefau Zefau mentioned this issue Jun 24, 2021
18 tasks
@Zefau
Copy link
Owner

Zefau commented Jul 19, 2021

bitte schau nochmal mit v3.0.0-alpha.32.

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Jul 19, 2021

Bei Leerwerten bleibt das Komma da:
image
Bei Nullwerten bleibt die Einheit und das Komma da:
image
Bei aufeinanderfolgenden Nullwerten kein Komma:
image

@Zefau
Copy link
Owner

Zefau commented Jul 19, 2021

bitte schau nochmal mit v3.0.0-alpha.33.

@mcuiobroker
Copy link
Collaborator Author

Leerwerte werden noch falsch angezeigt:
image
Dahinter steht Fehlergrund und der ist ein Text.
image

@Zefau
Copy link
Owner

Zefau commented Jul 19, 2021

Leerwerte werden raus gesammelt eigentlich. Ist das ein Leerzeichen oder was genau?

@mcuiobroker
Copy link
Collaborator Author

Da steht nichts drin, also null?

@Zefau
Copy link
Owner

Zefau commented Jul 19, 2021

hasValue(val) {
	return val !== undefined && val !== null && val !== '' && val !== 0 && val !== '0';
}

Das wird raus gesammelt

@Zefau
Copy link
Owner

Zefau commented Jul 21, 2021

mit v3.0.0-alpha.33 bleibt nur noch #791 (comment), aber die zwei aus #791 (comment) sind behoben, oder?

@mcuiobroker
Copy link
Collaborator Author

Ja , genauso war es auch schon mit v2.2.0

@Zefau
Copy link
Owner

Zefau commented Jul 21, 2021

Ok, also alter Zustand wiederhergestellt, aber noch nicht optimal?

Bei #791 (comment) handelt es sich um einen validen Wert, der ein Komma hat. Das sollte ich noch hinbekommen.

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Jul 21, 2021

Vielleicht hilft es in den Code von v2.2.0 zu schauen, wie du es dort gemacht hast?
Ist von diesem Gerät im Layout:
image

@Zefau
Copy link
Owner

Zefau commented Jul 21, 2021

Vielleicht hilft es in den Code von v2.2.0 zu schauen, wie du es dort gemacht hast?

Dachte der aktuelle Zustand entspricht dem aus v2.2.0?

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Jul 21, 2021

Nein. Die Komma Probleme sind dort behoben.

@Zefau
Copy link
Owner

Zefau commented Jul 21, 2021

Dann haben wir aneinander vorbei geredet. Ich hab es so gelesen, dass es auch in v3 behoben ist?

@Zefau
Copy link
Owner

Zefau commented Jul 21, 2021

Nur #791 (comment) ist noch offen, wobei es ja kein Leerwert ist, aber das Komma natürlich falsch gesetzt.

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Jul 21, 2021

Ja, Fehlergrund ist aber Leerwert hinter dem anzeigten Komma und CountDownRestSek ist ein 0-Wert.

@Zefau
Copy link
Owner

Zefau commented Jul 21, 2021

Schau nochmal mit v3.0.0-alpha.36.

@mcuiobroker
Copy link
Collaborator Author

Mach ich, wenn sie da ist.

@mcuiobroker
Copy link
Collaborator Author

v3.0.0-alpha.36
image
Jetzt fehlt ein Komma, wenn Fehlergrund gefüllt ist mit einem Text.

@mcuiobroker
Copy link
Collaborator Author

Auch mit .39:
image

@Zefau
Copy link
Owner

Zefau commented Aug 1, 2021

Wie ist der Status mit v3.0.0-alpha.46 ?

@mcuiobroker
Copy link
Collaborator Author

v.46 Kein Unterschied oben mit v.39. Komma bleibt hinter den 100% stehen.

@rschneider1509
Copy link

rschneider1509 commented Sep 3, 2021

Eine Frage/Anmerkung zum Thema, allerdings leicht offtopic; wenn gewünscht mache ich einen eigenen Fall auf:
Wenn man die sekundären Datenpunkte anders färbt, bleibt das Komma im standard Grau (schon in 2.2)

@Zefau
Copy link
Owner

Zefau commented Sep 13, 2021

@rschneider1509 korrekt wie durch @mcuiobroker dargestellt ist es in der Färbung inkludiert

@mcuiobroker
Copy link
Collaborator Author

Nicht ganz?
image

@Zefau
Copy link
Owner

Zefau commented Sep 13, 2021

oh, stimmt, ist auch laut Code mit in der Färbung inkludiert.

@Zefau
Copy link
Owner

Zefau commented Sep 13, 2021

schau nochmal bitte mit v3.0.0-alpha.76.

@mcuiobroker
Copy link
Collaborator Author

.76
image

@Zefau
Copy link
Owner

Zefau commented Sep 14, 2021

schau dir nochmal mit v3.0.0-alpha.77 an

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Sep 14, 2021

.77
Hier ist jetzt eine Einheit dazugekommen
image
Hier ist der Hintergrund an Werten wie oben schon gezeigt:
image
Hier ist die Einstellung für die Anzeigevarainte (countdownRestSek) aber 0 ausblenden -> Komma davor nicht anzeigen:
image
Und auch die Einheit wird ausgeblendet:
image


Nächster Fall:

image
Das Komma hat diesen Hintergrund an Werten, vorher stand immer 0,0 dort (was eigentlich richtig war, da man Nullwerte ja in der Anzeigevarainte ausblenden "0":"" soll v2.2.0 ):
image

@Zefau
Copy link
Owner

Zefau commented Sep 15, 2021

Okay, jetzt blick ich durch. Ich hab die ganze Zeit val (also den Rohwert). Ich stell es nun auf value um (also den verarbeiteten Wert mit display), dann sollte es passen.

@Zefau
Copy link
Owner

Zefau commented Sep 15, 2021

schau nochmal mit v3.0.0-alpha.78.

@mcuiobroker
Copy link
Collaborator Author

.78passt nocht nicht:
image
image

@mcuiobroker
Copy link
Collaborator Author

Also mit v3.0.0-alpha.79 passt es. Ich schließe den Punkt, falls noch etwas auffallen sollte, erstelle ich einen neuen.
Danke.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants