Skip to content

feat: Make gamepad status lighting optional#1705

Merged
encounter merged 12 commits into
TwilitRealm:mainfrom
superyu1337:feature/disable-gamepad-lighting
May 28, 2026
Merged

feat: Make gamepad status lighting optional#1705
encounter merged 12 commits into
TwilitRealm:mainfrom
superyu1337:feature/disable-gamepad-lighting

Conversation

@superyu1337

Copy link
Copy Markdown
Contributor

This creates an option for enabling/disabling the gamepad status lighting. The default is true, to not change behavior. But it allows a user to disable the option, should they wish to change the gamepad's lighting or turn it off entirely.

@MelonSpeedruns

Copy link
Copy Markdown
Collaborator

Maybe rename it to "Enable Controller LED" ? Sounds easier to understand

@superyu1337

Copy link
Copy Markdown
Contributor Author

Maybe rename it to "Enable Controller LED" ? Sounds easier to understand

Done

@MelonSpeedruns MelonSpeedruns left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@Abzol

Abzol commented May 21, 2026

Copy link
Copy Markdown
Contributor

Does this option only show when a LED is detected?

@superyu1337

Copy link
Copy Markdown
Contributor Author

Does this option only show when a LED is detected?

No, I could add that tho.

@MelonSpeedruns

Copy link
Copy Markdown
Collaborator

Does this option only show when a LED is detected?

No, I could add that tho.

Make sure it shows up and not when plugging in/out the controller.

@superyu1337 superyu1337 marked this pull request as draft May 23, 2026 16:01
@superyu1337

Copy link
Copy Markdown
Contributor Author

converted to draft until relevant PR in Aurora is merged and dusklight is updated.

@superyu1337 superyu1337 marked this pull request as ready for review May 23, 2026 16:19
Comment thread src/dusk/ui/bool_button.cpp Outdated
Comment thread src/dusk/gamepad_color.cpp Outdated
@superyu1337 superyu1337 requested a review from encounter May 27, 2026 13:22
encounter added 3 commits May 27, 2026 23:54
# Conflicts:
#	extern/aurora
#	include/dusk/settings.h
#	src/dusk/settings.cpp
@encounter encounter merged commit 71961da into TwilitRealm:main May 28, 2026
8 checks passed
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.

4 participants