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

New mode *Minutes* #238

Merged
merged 13 commits into from
Apr 25, 2023
Merged

Conversation

Wandmalfarbe
Copy link
Contributor

  • Der neue Modus kann auf der Seite Funktionen geändert werden und zeigt die Minuten als große Ziffern an, genau wie der Modus Sekunden.
  • Die Radio-Buttons und deren Label werden nicht mehr durch einen automatischen Zeilenumbruch voneinander getrennt.

  • The new mode can be changed on the page Functions and displays the minutes as big digits just like the mode Seconds.
  • The radio buttons and their labels are no longer separated from each other by an automatic line break.

mode-minutes

- The new mode can be changed on the page *Functions*
  and displays the minutes as big digits just like the mode *Seconds*
@dbambus
Copy link
Collaborator

dbambus commented Apr 19, 2023

Hallo @Wandmalfarbe,

Dies ist ein klasse Commit und eigentlich eine gute Erinnerung daran, was ich noch nicht implementiert habe. In dem Branch, den ich hier angehängt habe, gab es eine Digitaluhr. Die ich konzeptionell fertiggestellt hatte, die aber noch zwei Bugs bezüglich anderer Uhrengrößen hatte. Außerdem wollte ich zwei Features hinzufügen, um auch die Anzeige der IP-Adresse zu ermöglichen.

Ich werde versuchen, diese am Wochenende zu implementieren.

Hier würde ich eine neue kleinere Schriftgröße (5x3) für Zahlen und zwei Zeilen für Stunden und Minuten einführen.

Was hält du davon ?

This is a great commit and actually a good reminder of what I have not yet implemented. In the branch I attached here, there was a digital clock. Which I had conceptually finished, but which still had two bugs regarding other clock sizes. Furthermore, I wanted to add two features to also enable the display of the IP address.

I will try to implement it this weekend.

Here I would introduce a new smaller font size (5x3) for numbers and two lines for hours and minutes.

What do you think?

main...dbambus:Wortuhr:DigitalCLock

@Wandmalfarbe
Copy link
Contributor Author

Eine Digitaluhr ist natürlich eine wunderbare Ergänzung! Findest du, dass man dann die Modi Sekunden und Minuten ersetzen sollte? Ich denke, die Digitaluhr ist eine gute Ergänzung zu diesen beiden Modi.

@dbambus
Copy link
Collaborator

dbambus commented Apr 19, 2023

Ich denke das man Sekunden zumindest weiterbehalten sollte, eventuell als Countdown oder dergleichen.

Für Minuten sollte man dann zumindest eine Spezifizierung hinzufügen, aber da hab ich mich noch nicht ausreichend reingedacht.

Ich werde mal wichtige Punkte von deinem Commit übernehmen und dann die Implementierung von der Digitaluhr vervollständigen :-)

Bin aber noch auf Arbeit und kann da nichts testen :-)

@Wandmalfarbe
Copy link
Contributor Author

Ich denke das man Sekunden zumindest weiterbehalten sollte, eventuell als Countdown oder dergleichen.

Für Minuten sollte man dann zumindest eine Spezifizierung hinzufügen, aber da hab ich mich noch nicht ausreichend reingedacht.

Man kann ja dann einen Modus Digitaluhr einführen und die einzelnen Komponenten Stunden, Minuten, Sekunden in einem neuen Panel unter dem Panel Modus konfigurieren (so ähnlich wie auch bei der Laufschrift). Das ist natürlich deutlich mehr Arbeit im JavaScript, wegen Ein- und Ausblenden der Panels und übertragen der zusätzlichen Optionen an das Backend.

Ich werde mal wichtige Punkte von deinem Commit übernehmen und dann die Implementierung von der Digitaluhr vervollständigen :-)

Bin aber noch auf Arbeit und kann da nichts testen :-)

keinen Stress :)

@dbambus dbambus added the enhancement New feature or request label Apr 19, 2023
@dbambus
Copy link
Collaborator

dbambus commented Apr 20, 2023

Hallo @Wandmalfarbe,

Ich habe mal meinen Stand von damals auf diesen Pull Request überführt und gepushed. Falls du mal Zeit hättest.

Mit freundlichen Grüßen
David

@pienmatla
Copy link
Contributor

@dbambus I added a comment with the right Dutch translation. (Digital clock = Digitale klok). Would be helpful to change before merging

@Wandmalfarbe
Copy link
Contributor Author

@dbambus Wenn ich das bei mir aufspiele, kann ich im Webinterface zwar auf Digitaluhr schalten, aber es passiert nichts. Der korrekte Modus 7 wird aber per WebSocket gesendet. Ich habe sicherheitshalber einen Komplettreset gemacht und dann diesen Branch nochmal frisch aufgespielt.

@dbambus
Copy link
Collaborator

dbambus commented Apr 24, 2023

Hey @Wandmalfarbe,

Soweit scheint alles in Ordnung, noch letzte Anmerkungen vor dem Merge ?

Mit freundlichen Grüßen

@dbambus dbambus merged commit 3cabc6b into ESPWortuhr:main Apr 25, 2023
2 checks passed
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
Development

Successfully merging this pull request may close these issues.

None yet

3 participants