-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add region control #1
Comments
Hi @tomc98 , Thank you for your interest in this project, my own keyboard only has one color. Does it work for you? I haven't had the time to update the Python GUI yet. Feel free to fork and send me a pull request if you catch any bugs/ want to contribute |
Hey Kevin, so would it be difficult to make the package a .rpm? I know it's
an annoying but I've never built one to rpm before. If you can't don't
worry I'll figure it out.
Cheers.
…On Fri, Apr 26, 2019 at 6:36 AM Kevin Van Lierde ***@***.***> wrote:
Hi @tomc98 <https://github.com/tomc98> ,
Thank you for your interest in this project, my own keyboard only has one
color.
I've added support for the center, right and extra regions through the
bash CLI, see c156986
<c156986>,
and release 0.2
<https://github.com/webketje/tuxedo-backlight-control/releases/tag/v0.2>.
Does it work for you?
I haven't had the time to update the Python GUI yet. Feel free to fork and
send me a pull request if you catch any bugs/ want to contribute
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADYQKTFXBKQGDDNFOQGDC5LPSII5VANCNFSM4HHYXGZA>
.
--
Thomas Csere
ABN: 94252084235
Phone: 0423038799
|
I created a separate issue for this #2. Your feedback is welcome there and on this issue as well |
A lot of keyboards have the separate regions, you've only included the one. If you don't feel like doing it add me as contributor.
The text was updated successfully, but these errors were encountered: