Added Berndeutsch layout 11x10 + Sunset feature#51
Open
JCMengisen wants to merge 1 commit intotechniccontroller:mainfrom
Open
Added Berndeutsch layout 11x10 + Sunset feature#51JCMengisen wants to merge 1 commit intotechniccontroller:mainfrom
JCMengisen wants to merge 1 commit intotechniccontroller:mainfrom
Conversation
Added sunset darkmode (brightness 50%) instead of Nightmode(LED off) as LUT or polynomial
Owner
|
Hi Claude, vielen Dank für den PR. Wie ich sehe sind in dem PR vier Änderungen drin, die ich gerne etwas aufteilen würde:
Allgemein würde ich dich bitte meinen main branch vor eine PR einmal in deinen branch zu mergen, damit die neuen Änderungen mit drin sind. Viele Grüße |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added Berndeutsch layout 11x10
Added sunset darkmode (brightness 50%) instead of Nightmode(LED off) as LUT or polynomial
Hallo Edgar
Ich denke als Erstes ist es nicht die Idee, dass wir direkt in deinen Main branch mergen. Vielmehr wird es eher ein Review sein und anschliessend kann ich die neuen Features so anpassen, dass sie kompatibel sind zu deinem Konzept. Als Erstes habe ich den Darkmode einfach direkt in deinen NighMode integriert. Das kann man nautürlich trennen oder schöner machen, Wenn du dieses Feature sowieso nicht möchtest, lass ich es einfach auf meinem branch. Wir sehen! ;-)
=> Ich habe mir noch überlegt ob ich den PR auf draft stelle, bin mir aber nicht sicher ob du dann die Meldungen schon bekommst. => Du darfst es auch auf Draft zurück stellen!
Übrigens cooles Projekt und endlich ewas sinnvolles für die Arduinos!
Claude