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

atlas_65 Layout Macro Refactor #16483

Merged
merged 5 commits into from
Mar 1, 2022
Merged

Conversation

noroadsleft
Copy link
Member

Description

Flagged by /u/worldofindie on Reddit:

Programming works correctly when compiled from source, but out of sequence in Configurator.

Refactored the source because the Configurator data is as would be expected if the source conformed to QMK guidelines.

cc @ryanstevensmith (keyboard maintainer)

Types of Changes

  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)

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.
  • I have read the CONTRIBUTING document.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

- convert tabs to spaces
  - four-space indent
- align backslashes in layout macro
Visually separates each side. White-space-only change.
Move the matrix position identifiers in the layout macro to resemble the assembled keyboard's layout.

- move `k46` (right side B) to the fourth (Shift) row
- move each of `k1E`, `k2E` and `k3E` (right side navigation keys) up one row
- update keymaps to match
Update the maintainer data in `info.json` and `readme.md`.
@noroadsleft noroadsleft added enhancement keyboard keymap via Adds via keymap and/or updates keyboard for via support labels Mar 1, 2022
@noroadsleft noroadsleft requested a review from a team March 1, 2022 06:03
@drashna drashna requested a review from a team March 1, 2022 06:20
@noroadsleft noroadsleft merged commit 325c634 into qmk:master Mar 1, 2022
@noroadsleft noroadsleft deleted the cf/atlas_65 branch March 1, 2022 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement keyboard keymap via Adds via keymap and/or updates keyboard for via support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants