-
Notifications
You must be signed in to change notification settings - Fork 262
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
Color names (with and without underscores) #1309
Open
jdlien
wants to merge
11
commits into
keyboardio:master
Choose a base branch
from
jdlien:colors_underscores
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
A list of constexpr colors have been defined to match the names used in CSS, with a few minor tweaks where the colors look dramatically different on the Model 01. Signed-off-by: JD Lien <[email protected]>
This reverts commit aec5590. Signed-off-by: JD Lien <[email protected]>
Signed-off-by: JD Lien <[email protected]>
keygroups gives a variety of groups that can be used by plugins to determine if a given key object is within a particular group, such as alphabetical keys, number keys, modifiers, mouse movement keys, etc. This was developed originally for FunctionalColors. Signed-off-by: JD Lien <[email protected]>
Signed-off-by: JD Lien <[email protected]>
Added colors.h to have the CSS color names in the kaleidoscope::colors namespace. LEDEffect-SolidColor now has support for using cRGB objects via a new alternate constructor, and works with the color names. Keygroups.h has been added to offer functions that allow determining if a given key is in a certain group. Signed-off-by: JD Lien <[email protected]>
At @obra’s request, I have removed keygroups.h so that this PR can focus on colors. Signed-off-by: JD Lien <[email protected]>
Signed-off-by: JD Lien <[email protected]>
Signed-off-by: JD Lien <[email protected]>
Signed-off-by: JD Lien <[email protected]>
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.
This is an update (three years later) of #668
that adds the requested underscored names to the PR.
The non-underscored variants are included in a single section so they could easily be removed to reduce redundancy, but either way will not result in additional memory use.