Skip to content
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

fix: Fix inconsistent column offset property #2630

Merged

Conversation

joelspadin
Copy link
Collaborator

Renamed the composite kscan's column-offset property to col-offset for consistency with other properties such as matrix transform's col-offset and matrix kscan's col-gpios.

column-offset remains supported but is deprecated.

Renamed the composite kscan's column-offset property to col-offset for
consistency with other properties such as matrix transform's col-offset
and matrix kscan's col-gpios.
@joelspadin joelspadin requested review from a team as code owners November 10, 2024 02:52
Copy link
Contributor

@caksoylar caksoylar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Although trivial, I'll formally give an approval for the docs side.

@caksoylar caksoylar added the core Core functionality/behavior of ZMK label Nov 10, 2024
Copy link
Contributor

@petejohanson petejohanson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@petejohanson petejohanson merged commit c7473fc into zmkfirmware:main Nov 12, 2024
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core functionality/behavior of ZMK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants