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

Add QMK Configurator support and remove Layouts definitions #2977

Merged
merged 4 commits into from May 17, 2018

Conversation

mechmerlin
Copy link
Contributor

No description provided.

@drashna drashna changed the title Add QMK Configurator support and remove userspace definitions Add QMK Configurator support and remove Layouts definitions May 16, 2018
@drashna
Copy link
Member

drashna commented May 16, 2018

Userspace != Layouts. (was bothering me, sorry).

Also, why remove layout support for the keyboard here? Is there a specific reason?

If so, don't forget to edit the rules.mk to remove layout support.

@mechmerlin
Copy link
Contributor Author

I haven't removed any support. I've actually added a new layout

@drashna
Copy link
Member

drashna commented May 16, 2018

60_ansi/60_iso to ansi/iso?

@mechmerlin
Copy link
Contributor Author

Oh the renames? Yes I renamed them so people don't try to use userspace stuff without knowing about the unique RGB situation.

@drashna
Copy link
Member

drashna commented May 16, 2018

Ah. (also, "layouts", not "userspace"... that's going to bug me)

But I think it should still be part of the layout scheme. That's what conditional defines are for.

@jackhumbert, opinion?

@mechmerlin
Copy link
Contributor Author

mechmerlin commented May 17, 2018

Use standard layout but remove community layout support (Add a note about it) after conversation with @jackhumbert.

@drashna

@drashna
Copy link
Member

drashna commented May 17, 2018

Thanks!

@drashna drashna merged commit effc3e3 into qmk:master May 17, 2018
@mechmerlin mechmerlin deleted the configurator/hs60 branch May 19, 2018 14:49
carlpehrson pushed a commit to carlpehrson/qmk_firmware that referenced this pull request May 30, 2018
* Add QMK Configurator support and remove userspace definitions

* fix that build breakage

* Unsupport community layouts but use standard layouts
hauleth pushed a commit to hauleth/qmk_firmware that referenced this pull request Jan 24, 2019
* Add QMK Configurator support and remove userspace definitions

* fix that build breakage

* Unsupport community layouts but use standard layouts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants