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

[BUG] The "Use Camel Case" feature is semantically incorrect. #199

Open
stuboydl opened this issue Dec 24, 2024 · 0 comments
Open

[BUG] The "Use Camel Case" feature is semantically incorrect. #199

stuboydl opened this issue Dec 24, 2024 · 0 comments

Comments

@stuboydl
Copy link
Contributor

The operation of the "Use Camel Case" flag is semantically incorrect as multi-word keywords in the syntax files used are typically not in camel case.

(nb the descriptive text for the flag, "Use Camelcase for Intellisense keywords.", is heterological, as it uses Title case for the word whereas camelCase or CamelCase would be normatively autological).

To Reproduce
Enable the MVBasic: Use Camel Case flag in settings, type a keyword into the editing screen.

Keywords are displayed per entries in the syntax language files. Typically the multi-word entries have an initial capital letter followed by lower case regardless of any word boundary. Therefore, they are not in camel case.

eg UniverseLanguage.json

      {
        "key": "Deactivatekey",
        "icon": 14,
        "documentation": "Deactivate one or mor encryption keys.", [sic] 
        "detail": "DEACTIVATEKEY <key.id>, <password> [ON <hostname>]"
      },

image

Expected behavior
If the flag was enabled I would expect that keywords were displayed in camel case. For the above example, this would be "DeactivateKey" for upper camel case or "deactivateKey" for ordinary camel case.

Additional context
It might be more correct to reverse the use of the flag to a "Use Upper Case" flag (which is on by default). This would describe the operation of the server.ts code and makes no assumptions about the case specificity of the entries in the syntax/language files.

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

No branches or pull requests

1 participant