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

Fixes for choc option #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fixes for choc option #17

wants to merge 2 commits into from

Conversation

KnuthLohse
Copy link

The experimental choc option had some issues with the keyholes that I have addressed.

  • Keyholes have not been square due to an missing division by two in the keywall transformation.
  • Keyholes had MX size, although chocs are a bit smaller. Chocs are now set to 13.8mm square
  • Keyholes had an small heightoffset above the sourrounding plate, that has been fixed
  • Keyholesidewallheight (left/right) has been changed to fixed size of 2.2mm to allow the clamps of choc keys to grip on that wall.

open issue; For the 1.5u thumb buttons, the keyhole should be rotated 90 degrees to allow the rotated mount of the button for usage of horizontal keycaps instead of harder to source vertical ones (At least as option).

Tested the changes with the FDM print of this Manuform:
#manuform:Ch4IBhAEGgNzaXgiBHplcm8qBGNob2MyBW91dGllOAAaCggAEgRub25lGAAiF1UAAABBGAAgAV0AAOBAZQAAQEBAAEgAMvkBlQMAACBAnQMAAAAAgAMBiAMBDc3MTL4VAAAAAB1mZsZAJQAAsMAtzcxEwTUAAGBAPWZm5j9FmpmZPk2amblAVQAAQMBdAACgQGUzM3nCbTMzEcJ1AADQwXjCA4ABvxaIAdgTlQEzMy/CnQEzM03CpQEAAMjBqAGqBrABixW4AYgOxQEAAETCzQEAAMzB1QEAAHDB2AHQBeABixXoAbgI9QEAAOjB/QEAACTChQIAAGDBiALGCpAClRCYAvYEpQIAAAzCrQIAAHDBtQIAAKDAuAKEB8ACyRHIAoQH1QIAAHjB3QIAAGDB5QIAACBA6AKEB/AClRD4AsIDKgYIABAAGAA=

Beside the too low Vertical (Y) Spacing setting used, the Kailh Choc switches fit perfectly.

 - Fixed issue with non square key switch holes;
 - Reduced left (right) sidewall height of keyholes to fixed 2.2 for the clamps to grip;
 - fixed keyheight offset to be level with the surrounding plate.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants