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

Additional surface phrases. #116

Closed
inventor7777 opened this issue Jun 5, 2024 · 2 comments · Fixed by #117
Closed

Additional surface phrases. #116

inventor7777 opened this issue Jun 5, 2024 · 2 comments · Fixed by #117

Comments

@inventor7777
Copy link

inventor7777 commented Jun 5, 2024

Hello again!

I’ve been making a phrase wish list while using my radio, and I have some more phrase suggestions.

They are:

“Drift Mode Enabled”
“Drift Mode Disabled” (I know there are ones that sound similar, more on that in a sec)
“Screenshot”
“Cell” (for including in prompts about cell voltage)
“Transmitter Battery Low”
“Left”
“Right”
“Shift”
“Stability Control On”
“Stability Control Off”
“Stability Control Enabled”
“Stability Control Disabled”
“Thermal Warning” (“thermal” and “warning” already exist, so it would be there just to make the trigger sequence for the phrase simpler)

Now, as I’ve been selecting phrases, I’ve noticed something interesting: some phrases have an On/Off end, and some have Enabled/Disabled or even Active instead (see phrases like “course hold…” or “rescue mode…”. So I’ve added some phrases like “drift mode enabled/disabled” to pair well with other lines like “wet mode enabled/disabled”

in the future it’s probably something to look out for, to ensure integrated, smooth phrase transitions.

@pfeerick
Copy link
Member

pfeerick commented Jun 6, 2024

or even Active instead

They were probably deliberately done that way so the last word wasn't similar, so less of a chance of mishearing which one it said. This is why there ends up being a couple of different versions of them though, to suit preferences.

“Transmitter Battery Low” is already present as a system file... you want to be able to play that via special function rather than let the radio manage when it should be played?

Rest all makes sense though.

@inventor7777
Copy link
Author

inventor7777 commented Jun 6, 2024

you want to be able to play that via special function rather than let the radio manage when it should be played?

Exactly. I just want the option for it to warn me once at, say, 6.8v, so I have plenty of time to charge before it starts vibrating and yelling at me 😂. (I do know that I can change the alert time in the settings, but it will still constantly vibrate and yell at the set voltage....unless I'm missing something? My radio is usually charged so it's only happened once)

They were probably deliberately done that way so the last word wasn't similar, so less of a chance of mishearing which one it said. This is why there ends up being a couple of different versions of them though, to suit preferences.

As for Active, I totally get it (since when does "launch control enabled" sound cool?). I just think for some files both options (enabled/disabled and on/off) should be available.

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 a pull request may close this issue.

2 participants