fix: check for invalid speedometer names

[Trainermitch]

Deleting a speedometer with the same name as an existing speedometer results in an underlying object js exception. This is because this.detailObject.findIndex() inside deleteSpeedometer() deletes the first speedometer matching the name of the selected speedometer. I have changed speedometer-select.js to check for these invalid names (mostly used the same code from range-color-profile-name.js). This also makes it consistent with deleting and adding range profiles.

Checks

• I have thoroughly tested all of the code I have modified/added/removed to ensure something else did not break
• I have followed semantic commit messages e.g. feat: Add foo, chore: Update bar, etc...
• My branch has a clear history of changes that can be easy to follow when being reviewed commit-by-commit
• My branch is functionally complete; the only changes to be done will be those potentially requested in code review
• All changes requested in review have been fixuped into my original commits.
• Fully tokenized all my strings (no hardcoded English strings!!) and supplied bulk JSON strings below

POEditor JSON Strings (if needed)

[
	{
		"term": "#Settings_Speedometer_InvalidName",
		"definition": "Invalid speedometer name!"
	}
]

[tsa96]

LGTM

[Trainermitch]

Also, do I just need to supply the tokenized string or do I need to actually add it somewhere?

[tsa96]

Also, do I just need to supply the tokenized string or do I need to actually add it somewhere?

No just the token and string, whoever approves it will add it to POEditor (requires POEditor admin)

[tsa96]

Looks good. @Gocnak I've added the string to POEditor already, merge when you're happy!

[Gocnak]

Sweet, good stuff