Skip to content

Enable headlights button based on LEDs state - #115

Closed
OakCityDirtyRandy wants to merge 1 commit into
lukash:mainfrom
OakCityDirtyRandy:main
Closed

OakCityDirtyRandy wants to merge 1 commit into
lukash:mainfrom
OakCityDirtyRandy:main

Conversation

@OakCityDirtyRandy

Copy link
Copy Markdown
Contributor

Fix: Disable the Headlights button unless LEDs are On.

Fix:  Disable the Headlights button unless LEDs are On.
@lukash

lukash commented Oct 17, 2026

Copy link
Copy Markdown
Owner
Fix:  Disable the Headlights button unless LEDs are On.

You've got two spaces after the "Fix:" and by convetion there are no full stops at the end of the thing. I wanted to fix this in your PR but you've created it from main so I can't do that, so I just pushed the fixed commit manually.

Only after the fact I noticed there's also a double space in the actual changed line:

enabled:  lights.on

There's no linter on the QML. Please try to pay attention, I don't want messy code.

@lukash lukash closed this Oct 17, 2026
@lukash

lukash commented Oct 17, 2026 •

Copy link
Copy Markdown
Owner

Oh, I also decided to change the Fix: to Feature:, as it's not a fix per se, it's more like a tiny feature.

@OakCityDirtyRandy

OakCityDirtyRandy commented Oct 17, 2026 via email

Copy link
Copy Markdown
Contributor Author

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.

2 participants