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

RSSI Model Locator: Add TX Power & Haptic Feedback #177

Merged
merged 6 commits into from
Dec 15, 2024

Conversation

AegisCrusader
Copy link
Contributor

@AegisCrusader AegisCrusader commented Nov 28, 2024

This change adds the current TX power so that people can see if they are on 25mW or not when using the locator. Additionally adds haptic feedback for those hard of hearing or those who just like it

Adds the current TX power so that people can see if they are on 25mW or not
* Now uses haptic feedback
* Text has been moved to not overlap
@AegisCrusader AegisCrusader changed the title Add TX Power to Model Locator Add TX Power & Haptic Feedback to Model Locator Nov 28, 2024
@AegisCrusader AegisCrusader changed the title Add TX Power & Haptic Feedback to Model Locator RSSI Model Locator: Add TX Power & Haptic Feedback Nov 28, 2024
0.5 -> 0.6
@AegisCrusader AegisCrusader marked this pull request as ready for review November 28, 2024 17:13
@AegisCrusader
Copy link
Contributor Author

@offer-shmuely Do you have any preference for the version number change?

@offer-shmuely
Copy link
Contributor

offer-shmuely commented Dec 7, 2024

the changes looks ok
the idea is great, thank you
the version is ok as well.
I would hide the "Note: Set tx power to 25mW non dynamic" if the power is already 25mw
show it, only when the user need to really change it

also, the haptic default should be false

- Haptic now disabled by default
- Set power message disappears if the power is correct
@AegisCrusader
Copy link
Contributor Author

AegisCrusader commented Dec 8, 2024

Changes have been made! Haptic is disabled by default and the message disappears when 25mW is the current power.

However has not been tested for screen layout, etc
@pfeerick
Copy link
Member

LGTM... thank you for your contribution! :)

@pfeerick pfeerick merged commit ddededf into EdgeTX:master Dec 15, 2024
2 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.

3 participants