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

Nachkomma stelle anzeigen. #24

Closed
torretx opened this issue May 21, 2021 · 11 comments · Fixed by #25 or #31
Closed

Nachkomma stelle anzeigen. #24

torretx opened this issue May 21, 2021 · 11 comments · Fixed by #25 or #31
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@torretx
Copy link

torretx commented May 21, 2021

Da einige Regionen nun an diversen Schwellen sind (50 & 100) wäre es toll wenn die App eine nachkomma stelle anzeigen würde.

Bei mir z.b sind wir heute auf 49,7 das Widget zeigt aber 50 an. Da ein Landkreis mehrere Tage unter 50 sein muss damit Lockerungen greifen, wäre die Anzeige kontraproduktiv im Widget.

Oder man verzichtet auf das runden und lässt einfach die nachkomma stelle weg.

@marcusraitner marcusraitner self-assigned this May 21, 2021
@marcusraitner marcusraitner added the enhancement New feature or request label May 21, 2021
@marcusraitner
Copy link
Owner

Ja, da gebe ich dir recht. Kommastelle ist nicht so einfach, weil zu wenig Platz, aber Abrunden wäre sicher eine Variante, die mehr Sinn macht als das heutige Verhalten.

@Drail0r
Copy link

Drail0r commented May 21, 2021

Moin,
eine Idee: du könntest eventuell den Impfstatus etwas schmaler machen, dort die Nachkommastelle entfernen und die Schriftgröße verkleinern und könntest so Pixel gewinnen und zumindest den aktuellen Tag, das Bundesland und Deutschland mit einer Nachkommastelle darstellen lassen. Fände ich zumindest schöner, als mathematisch inkorrekt abzurunden.

@marcusraitner
Copy link
Owner

Moin,

eine Idee: du könntest eventuell den Impfstatus etwas schmaler machen, dort die Nachkommastelle entfernen und die Schriftgröße verkleinern und könntest so Pixel gewinnen und zumindest den aktuellen Tag, das Bundesland und Deutschland mit einer Nachkommastelle darstellen lassen. Fände ich zumindest schöner, als mathematisch inkorrekt abzurunden.

Eine gute Idee. Die Nachkommastelle beim Impfen braucht es jetzt nicht mehr; das ist eher historisch so gewachsen, weil damals sehr kleine Werte. Ich schau mir das nochmals an.

@marcusraitner marcusraitner reopened this May 21, 2021
@marcusraitner
Copy link
Owner

marcusraitner commented May 21, 2021

Wenn ich die Nachkommastellen anzeige, dann würde da etwa so aussehen, hätte aber zur Folge, dass man weniger Tage sieht (konkret drei Tage weniger). Sähe dann so aus:

IMG_1031

Evtl. sollte ich das parametrisierbar machen.

@Drail0r: Was meinst du?

@Drail0r
Copy link

Drail0r commented May 21, 2021

Kannst du nur die letzten drei Werte, also im Bild den 20., BY und DE mit Nachkommastellen anzeigen? Oder wenn es nach der Bundesnotbremse geht, dann eben die letzten 3 Tage? Weiter zurückliegende Tage würde ich korrekt gerundet anzeigen lassen.

Achso: Parametrisierbarkeit ist immer gut!

@marcusraitner
Copy link
Owner

Lässt sich sicherlich machen, aber ich glaube, dass dann sehr inkonsistent und unruhig wirkt. Ich glaube, ich mach es lieber so und dann parametrisierbar.

@Drail0r
Copy link

Drail0r commented May 21, 2021

Alles klar! Danke! ♥

@marcusraitner
Copy link
Owner

@Drail0r Probier' mal mit dem aktuellen Stand decimal=y

@Drail0r
Copy link

Drail0r commented May 21, 2021

Funktioniert und sieht fast perfekt aus. Ich finde es optisch schöner, wenn bei runden Werten auch xx.0 dargestellt wird. Aktuell wird dort nur die Zahl ohne Komma und Nachkommastellen angezeigt.

@marcusraitner
Copy link
Owner

Guter Punkt. Jetzt sollte es passen.

@Drail0r
Copy link

Drail0r commented May 21, 2021

Gute Arbeit! Vielen Dank! <3

@marcusraitner marcusraitner added this to the v1.7.0 milestone May 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
3 participants