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

Nebula12b #17308

Merged
merged 5 commits into from
Jul 1, 2022
Merged

Nebula12b #17308

merged 5 commits into from
Jul 1, 2022

Conversation

yiancar
Copy link
Contributor

@yiancar yiancar commented Jun 4, 2022

Description

Successor of Nebula12.
Similar story as Nebula68b:)

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

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.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@github-actions github-actions bot added keyboard keymap via Adds via keymap and/or updates keyboard for via support labels Jun 4, 2022
@drashna drashna requested a review from a team June 7, 2022 22:19
@yiancar
Copy link
Contributor Author

yiancar commented Jun 7, 2022

Ah yes nice!
One addition for the via code tho (in via.c) would be to scale brightness with MAX_BRIGHTNESS variable ;) I can do the PR if you like.

yiancar and others added 2 commits June 8, 2022 00:40
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
keyboards/spaceholdings/nebula12b/readme.md Outdated Show resolved Hide resolved
keyboards/spaceholdings/nebula12b/nebula12b.c Outdated Show resolved Hide resolved
@drashna
Copy link
Member

drashna commented Jun 8, 2022

Ah yes nice! One addition for the via code tho (in via.c) would be to scale brightness with MAX_BRIGHTNESS variable ;) I can do the PR if you like.

Hmm, I'm not sure. In theory, even if you set the val to a higher value, it won't use past that. But I'm not sure that via will re-read and report that correctly? May need some testing/etc.

yiancar and others added 2 commits June 8, 2022 08:42
upsie

Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
@yiancar
Copy link
Contributor Author

yiancar commented Jun 8, 2022

Ah yes nice! One addition for the via code tho (in via.c) would be to scale brightness with MAX_BRIGHTNESS variable ;) I can do the PR if you like.

Hmm, I'm not sure. In theory, even if you set the val to a higher value, it won't use past that. But I'm not sure that via will re-read and report that correctly? May need some testing/etc.

Yes that is correct, essentially by doing a scale operation (as I had done here) the brightness dial will not jump around in via. Currently lets say max brightness is at 150, half the dial does nothing. You slide it to 100% and it will jump back to 50% after the refresh. I had quite a few users getting confused about that!

@drashna drashna requested a review from a team June 10, 2022 01:12
@yiancar
Copy link
Contributor Author

yiancar commented Jun 30, 2022

Can we get this merged please?:)

@drashna drashna merged commit 1e643ba into qmk:master Jul 1, 2022
@drashna
Copy link
Member

drashna commented Jul 1, 2022

Thanks!

0xcharly pushed a commit to Bastardkb/bastardkb-qmk that referenced this pull request Jul 4, 2022
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: yiancar <yiancar@gmail.com>
@yiancar yiancar deleted the nebula12b branch July 18, 2022 21:33
schattenbrot pushed a commit to schattenbrot/qmk_firmware that referenced this pull request Aug 2, 2022
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: yiancar <yiancar@gmail.com>
nolanseaton pushed a commit to nolanseaton/qmk_firmware that referenced this pull request Jan 23, 2023
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: yiancar <yiancar@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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