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

Ergoarrows info.json touch-up #13537

Merged
merged 3 commits into from
Jul 13, 2021
Merged

Conversation

noroadsleft
Copy link
Member

Description

Removes dead space from the Configurator rendering, and fixes some rounding errors in key positioning.

cc @Salicylic-acid3 (keyboard maintainer)

Types of Changes

  • Enhancement/optimization
  • Keyboard (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).

Add line breaks between rows and halves.
Fixes issues with y-offset values due to rounding in KLE.
Removes the empty white space from the layout, and aligns the keys on the right half so they are a mirror of the left half.
@noroadsleft noroadsleft merged commit 555a449 into qmk:master Jul 13, 2021
@noroadsleft noroadsleft deleted the cf/ergoarrows branch July 13, 2021 20:44
nhongooi pushed a commit to nhongooi/qmk_firmware that referenced this pull request Dec 5, 2021
* human-friendly formatting

Add line breaks between rows and halves.

* fix rounding issues

Fixes issues with y-offset values due to rounding in KLE.

* remove layout dead space; re-mirror halves

Removes the empty white space from the layout, and aligns the keys on the right half so they are a mirror of the left half.
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.

3 participants