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

K2 layout fixes #67

Draft
wants to merge 7 commits into
base: sn32
Choose a base branch
from
Draft

K2 layout fixes #67

wants to merge 7 commits into from

Conversation

enricorov
Copy link

@enricorov enricorov commented Jul 31, 2021

Description

Restore stock media and volume fn key behaviour. Added hue/saturation controls for RGB on the default fn layer.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@enricorov enricorov changed the title Fixed K2 media keys. Added RGB hue/sat to fn layer K2 layout fixes Jul 31, 2021
@dexter93
Copy link

dexter93 commented Aug 1, 2021

Custom keymaps should not be committed in the default one. If you'd like your custom one to be merged, please name it appropriately in a new keymap. Space Cadet Shift and key swapping have no place in the default. Default keymaps should be as the manufacturer released them

@CalcProgrammer1 CalcProgrammer1 force-pushed the sn32 branch 2 times, most recently from f4d5809 to 7148084 Compare September 4, 2021 06:00
@CalcProgrammer1 CalcProgrammer1 force-pushed the sn32 branch 15 times, most recently from e386379 to 95168fa Compare October 3, 2021 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants