-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[Nanoleaf] Add new channel: State #13746
Conversation
Shows an image of the state of the panels with color. Also makes the layout slightly prettier. This is less functional than the layout, and more eyecandy. Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That review was a huge one - I hope the next ones are going to be smaller ;-)
...but the outcome is worth taking the time.
Mainly it looks good to me: there only minor comments from me.
@jlaur after @austvik has done the small amendments I recommend you merge it (if you don't have the time to fully review it).
...ain/java/org/openhab/binding/nanoleaf/internal/layout/shape/BarycentricTriangleGradient.java
Show resolved
Hide resolved
...ain/java/org/openhab/binding/nanoleaf/internal/layout/shape/BarycentricTriangleGradient.java
Show resolved
Hide resolved
...inding.nanoleaf/src/main/java/org/openhab/binding/nanoleaf/internal/layout/shape/Square.java
Outdated
Show resolved
Hide resolved
- Example in README - Renames to make the Panels/Shapes easier to understand Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
...binding.nanoleaf/src/main/java/org/openhab/binding/nanoleaf/internal/layout/shape/Shape.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Thanks, Stefan. Reworded the comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent, thanks! Only a minor comment about README formatting.
Split README in lines to make easier changes later. Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Co-authored-by: Jacob Laursen <jacob-github@vindvejr.dk>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* [Nanoleaf] New Channel: State Shows an image of the state of the panels with color. Also makes the layout slightly prettier. This is less functional than the layout, and more eyecandy. Signed-off-by: Jørgen Austvik <jaustvik@acm.org> Signed-off-by: Ben Rosenblum <rosenblumb@gmail.com>
* [Nanoleaf] New Channel: State Shows an image of the state of the panels with color. Also makes the layout slightly prettier. This is less functional than the layout, and more eyecandy. Signed-off-by: Jørgen Austvik <jaustvik@acm.org> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
* [Nanoleaf] New Channel: State Shows an image of the state of the panels with color. Also makes the layout slightly prettier. This is less functional than the layout, and more eyecandy. Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
* [Nanoleaf] New Channel: State Shows an image of the state of the panels with color. Also makes the layout slightly prettier. This is less functional than the layout, and more eyecandy. Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
* [Nanoleaf] New Channel: State Shows an image of the state of the panels with color. Also makes the layout slightly prettier. This is less functional than the layout, and more eyecandy. Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
* [Nanoleaf] New Channel: State Shows an image of the state of the panels with color. Also makes the layout slightly prettier. This is less functional than the layout, and more eyecandy. Signed-off-by: Jørgen Austvik <jaustvik@acm.org> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
This is an addition that shows an image of the state of the panels with color.
Also makes the layout slightly prettier. This is less functional than the layout, and more eye-candy.
Idea by Stefan Hoehn from #13552
Signed-off-by: Jørgen Austvik jaustvik@acm.org