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

Adding 1u center column layout to Promenade (HS) #2404

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

Feags
Copy link
Contributor

@Feags Feags commented Jan 19, 2025

Description

The Promenade comes in two layouts: 1.5u center column and 1u center column. It is 1.5u by default. The 1u and 1.5u pcbs are exactly the same, only differentiated by which key locations have hotswap sockets. The original firmware for the Promenade only had the 1.5u center layout. This change adds 1u layout option.

QMK Pull Request

Original Promenade Firmware added
Promenade Layout updated

VIA Keymap Pull Request

the-via/qmk_userspace_via#52

Checklist

  • The VIA support for this keyboard is MERGED in QMK master already (MANDATORY)
  • VIA keymap is MERGED in VIA userspace master already (MANDATORY)
  • The VIA definition follows the guide here: https://caniusevia.com/docs/layouts
  • I have a V3 JSON version for this keyboard definition.(MANDATORY)
  • I have formatted the JSON file to have consistent formatting with the rest of the repository.
  • I have tested this keyboard definition using VIA's "Design" tab.
  • I have tested this keyboard definition with firmware on a device.
  • I have assigned alpha keys and modifier keys with the correct colors.
  • The Vendor ID is not 0xFEED

@Feags Feags marked this pull request as ready for review February 3, 2025 07:20
@Cipulot
Copy link
Collaborator

Cipulot commented Feb 6, 2025

@Feags I suggest pulling the latest changes of the repo and properly formatting only the file you are targeting. this PR seems to change the formatting of 2 other JSON unrelated to the PR.

@Cipulot Cipulot added the needs work The PR needs fixes/ things need to be addressed label Feb 6, 2025
@Feags Feags force-pushed the kradoindustries-dev-branch branch from 529013c to 950d077 Compare February 9, 2025 00:58
@Feags
Copy link
Contributor Author

Feags commented Feb 9, 2025

@Feags I suggest pulling the latest changes of the repo and properly formatting only the file you are targeting. this PR seems to change the formatting of 2 other JSON unrelated to the PR.

Merged from master and removed the other 2 files. Should just be the one now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs work The PR needs fixes/ things need to be addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants