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

urbanvanilla keymap for drop/alt #11365

Merged
merged 22 commits into from
Jan 14, 2021
Merged

Conversation

urbanvanilla
Copy link

@urbanvanilla urbanvanilla commented Dec 30, 2020

Description

Keymap with simple changes all in userspace for the drop/alt.
Modified matrix.c in the wait_us function to get rid of annoying key chatter that plagues this keyboard.
Implemented a really simple RGB timer that turns off RGB Matrix lighting after a set time.
Changed escape key to grave escape.

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).

@drashna drashna requested a review from a team January 7, 2021 07:00
Copy link
Author

@urbanvanilla urbanvanilla left a comment

Choose a reason for hiding this comment

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

made the changes as reviewed, except for the matrix.c not sure if I should clear it all and just have the one function or keep it all with that minor change.

Copy link
Member

@zvecr zvecr left a comment

Choose a reason for hiding this comment

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

With #11472, matrix,.c can be removed and the suggestion applied to get the same behaviour.

@github-actions github-actions bot added cli qmk cli command core documentation keyboard python translation via Adds via keymap and/or updates keyboard for via support labels Jan 8, 2021
@drashna
Copy link
Member

drashna commented Jan 11, 2021

This has a bunch of existing commits, that is messing up the diff and such.
It looks like this may need a rebase to clean stuff up.

Urbanvanilla's keymap for massdrop/alt keyboard
adds in an idle timer to turn off RGB after set amount of time.
poor unsuccessful attempt at replicating a velocikey like functionality.
Grave escape is enabled on ESC.
removed unnecessary lines and files.
increased wait_us() parameter to 40 as was still getting chatter at 20.
removed some code for the terrible rudimentary attempt at velocikey for rgb matrix
This reverts commit deb55d7, reversing
changes made to ba88173.
Urbanvanilla's keymap for massdrop/alt keyboard
adds in an idle timer to turn off RGB after set amount of time.
poor unsuccessful attempt at replicating a velocikey like functionality.
Grave escape is enabled on ESC.
removed unnecessary lines and files.
increased wait_us() parameter to 40 as was still getting chatter at 20.
removed some code for the terrible rudimentary attempt at velocikey for rgb matrix
@github-actions github-actions bot removed translation documentation python core via Adds via keymap and/or updates keyboard for via support cli qmk cli command labels Jan 11, 2021
@drashna drashna merged commit 214528b into qmk:master Jan 14, 2021
@drashna
Copy link
Member

drashna commented Jan 14, 2021

Thanks!

1 similar comment
@urbanvanilla
Copy link
Author

Thanks!

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.

4 participants