Skip to content

Add options for vertical, extra led and meander layout #582

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

Conversation

BastiMu
Copy link
Contributor

@BastiMu BastiMu commented Apr 23, 2025

Ich habe noch ein paar Optionen hinzugefügt, damit das Layout unabhängig von der Bauform ist.

Extra LED bezieht sich auf die Lötfaule Option mit denen ich gerne die Uhren baue:
PXL_20250423_172741165

@dbambus
Copy link
Collaborator

dbambus commented Apr 23, 2025

Hey,

das muss ich mal genauer testen. Kannst du noch den Clang Formatter zufrieden stellen ?

Kleine Frage, hast du auch ein Problem mit dem libFrameWorkArduino.a ? Mein Platformio will einfach nicht bauen.

Mit freundlichen Grüßen
David

@BastiMu
Copy link
Contributor Author

BastiMu commented Apr 24, 2025 via email

@dbambus
Copy link
Collaborator

dbambus commented Jun 6, 2025

Hey @BastiMu,

vielen Dank für deinen Beitrag. Das wird der erste der offenen Pull Requests sein, den ich Merge. :-) muss das etwas abstimmen, da ich es noch testen will.

Mit freundlichen Grüßen
David

@dbambus dbambus merged commit ef1ac0b into ESPWortuhr:main Jun 6, 2025
2 checks passed
@dbambus
Copy link
Collaborator

dbambus commented Jun 6, 2025

Nice,

funktioniert wie erwartet. Jetzt müssten wir uns ggf nur mal der UI annehmen. Was hältst du von Pictogrammen für die verschiedenen Einstellungen?

Mit freundlichen Grüßen
David

@BastiMu BastiMu deleted the Add-options-for-vertical,-extra-led-and-meander-layout branch June 11, 2025 16:35
@BastiMu
Copy link
Contributor Author

BastiMu commented Jun 11, 2025

Pictogramme sind ne gute Idee, allerdings ist mir nicht ganz klar, wie man die einpflegen könnte.

Und zurzeit hab ich leider nicht übermäßig Zeit...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants