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

Dimple: fix unintended LED behaviour #6046

Merged
merged 2 commits into from
Jun 3, 2019
Merged

Dimple: fix unintended LED behaviour #6046

merged 2 commits into from
Jun 3, 2019

Conversation

Erovia
Copy link
Member

@Erovia Erovia commented Jun 1, 2019

The LED was always-on if the custom keymap did not call dimple_led_off()
at least once.

Description

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.
  • 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).

The LED was always-on if the custom keymap did not call dimple_led_off()
at least once.
keyboards/lazydesigners/dimple/dimple.c Outdated Show resolved Hide resolved
keyboards/lazydesigners/dimple/dimple.c Outdated Show resolved Hide resolved
keyboards/lazydesigners/dimple/dimple.c Outdated Show resolved Hide resolved
@Erovia
Copy link
Member Author

Erovia commented Jun 3, 2019

@drashna thanks for the suggestions. The new commit should contain them.

Copy link
Member

@drashna drashna left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

@drashna drashna merged commit a678f4a into qmk:master Jun 3, 2019
Timbus pushed a commit to Timbus/qmk_firmware that referenced this pull request Jun 23, 2019
* Dimple: fix unintended LED behaviour

The LED was always-on if the custom keymap did not call dimple_led_off()
at least once.

* Dimple: LED code fixup
@Erovia Erovia deleted the dimple branch October 6, 2019 11:30
ridingqwerty pushed a commit to ridingqwerty/qmk_firmware that referenced this pull request Jan 10, 2020
* Dimple: fix unintended LED behaviour

The LED was always-on if the custom keymap did not call dimple_led_off()
at least once.

* Dimple: LED code fixup
JeffreyPalmer pushed a commit to JeffreyPalmer/qmk_firmware that referenced this pull request Feb 27, 2020
* Dimple: fix unintended LED behaviour

The LED was always-on if the custom keymap did not call dimple_led_off()
at least once.

* Dimple: LED code fixup
swamp09 pushed a commit to swamp09/qmk_firmware that referenced this pull request Mar 11, 2020
* Dimple: fix unintended LED behaviour

The LED was always-on if the custom keymap did not call dimple_led_off()
at least once.

* Dimple: LED code fixup
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.

2 participants