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

Lotus 58, Vial keymap #591

Merged
merged 13 commits into from
Nov 12, 2023
Merged

Lotus 58, Vial keymap #591

merged 13 commits into from
Nov 12, 2023

Conversation

TweetyDaBird
Copy link
Contributor

@TweetyDaBird TweetyDaBird commented Nov 7, 2023

Adding Vial keymap, updated to latest QMK keycodes and moved appropriate settings to info.JSON

BTW, the two options failing in compile is due to lack of space using default bootloaders, and aren't viable as targets for Vial. So either these can be ignored, or someone can show me how to 'turn them off'?

Replacing key-codes to match latest QMK iteration, some options are moved to .JSON
@TweetyDaBird TweetyDaBird marked this pull request as draft November 7, 2023 12:04
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
Copy link
Contributor

@lesshonor lesshonor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot to mention this in my original comment: if you really only want to provide this keymap for nanoboot, you can put it into a keymaps folder under nanoboot—i.e., keyboards/tweetydabird/lotus58/nanoboot/keymaps/vial.
But if I was you I would just selectively disable more features in the rules.mk. E.g.

# Reduce size on atmega32u4
ifeq ($(strip $(MCU)), atmega32u4)
    TAP_DANCE_ENABLE = no
    QMK_SETTINGS = no
    KEY_OVERRIDE_ENABLE = no
    COMBO_ENABLE = no
    # Reduce size even more for non-nanoboot bootloaders
    ifneq ($(strip $(BOOTLOADER_SIZE)), 512)
        MAGIC_ENABLE = no
        MOUSEKEY_ENABLE = no
    endif
endif

keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c Outdated Show resolved Hide resolved
@TweetyDaBird
Copy link
Contributor Author

Yep, working on it and testing.

Unfortunately a trip to the vet with the family cat got in between. Feeling a bit low right now, so I'll pick up later.

@TweetyDaBird TweetyDaBird marked this pull request as ready for review November 11, 2023 15:03
Copy link
Contributor

@lesshonor lesshonor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The keyboard-level change here is quite reasonable as the same change is being made in an in-progress QMK PR.

Comment on lines +2 to +4
"name": "Lotus 58 Glow VIAL",
"author": "Markus Knutsson",

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"name": "Lotus 58 Glow VIAL",
"author": "Markus Knutsson",

Technically not necessary for Vial.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, not necessary, but very beneficial as it shows as the device name in most OS's. Meaning it's easy to identify if a keyboard is flashed with vial or QMK.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That isn't determined by vial.json, but by info.json. The Vial GUI concatenates manufacturer with keyboard_name.

I flashed an Elite-Pi with this PR.

(I hope your cat is doing better. If that isn't the case, my sympathies.)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm. Ok. Then we are editing that.

No, unfortunately after 17 years of life, out of which 8 being my daughters life long companion/slash teddy bear/snuggle cat, she past. So yeah, cry fest.

@xyzz xyzz merged commit 966f801 into vial-kb:vial Nov 12, 2023
3 checks passed
zhol0777 pushed a commit to zhol0777/vial-qmk that referenced this pull request Nov 20, 2023
* adding vial id

* Update Vial keymap

Replacing key-codes to match latest QMK iteration, some options are moved to .JSON

* Update keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* Update keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* Update keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* Update keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* Simplified OLED, cleaned up keymap

Simplified OLED, cleaned up keymap

* Update keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* Update keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* Update keyboards/tweetydabird/lotus58/keymaps/vial/keymap.c

Co-authored-by: Less/Rikki <[email protected]>

* conditional

---------

Co-authored-by: Less/Rikki <[email protected]>
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.

3 participants