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

Syntax highlighting breaks after using function with string containing colon #16

Open
MeliusCZ opened this issue May 5, 2024 · 4 comments
Labels
bug Something isn't working good first issue Good for newcomers requires more testing "I didn't manage to replicate this issue, so more science is required"

Comments

@MeliusCZ
Copy link

MeliusCZ commented May 5, 2024

When utilizing any function and inputting a string containing a colon (":"), the syntax highlighting for the remainder of the code breaks.
The syntax highlighting should remain intact for the entirety of the code after utilizing the function, irrespective of the presence of a colon in the string.

Steps to Reproduce:

  1. Use the function with a string containing a colon, like so: EventHandler.SendNetworkEvent("giveitem:BFG9000");
  2. Observe the syntax highlighting behavior in the subsequent code.
@KaptainMicila
Copy link
Owner

Thanks for the issue notice. I'll fix it once I can.
I've probably missed something in the strings handling.

@KaptainMicila KaptainMicila self-assigned this May 5, 2024
@KaptainMicila KaptainMicila added bug Something isn't working good first issue Good for newcomers labels May 5, 2024
@KaptainMicila
Copy link
Owner

This will be a funny response: I can't re-create the issue. I tried copy-pasting your piece of code in a few places to see if there was some kind of edge-case issue, but I can't get the syntax highlighting to break.

@MeliusCZ
Copy link
Author

MeliusCZ commented Jul 7, 2024

Thanks for checking, but it happens to me every time, here is what it looks like:
Issue

@KaptainMicila
Copy link
Owner

From the picture you've sent me it seems that the highlighting was broken way before the first double quote.
So, I'll ask you:

  • May you send me the same screenshot, but with the default vscode theme?
  • Are you using any string escaping in the code (there's an issue open about this at time of writing)
  • Is it possible to receive a bigger snippet of code?

@KaptainMicila KaptainMicila added the requires more testing "I didn't manage to replicate this issue, so more science is required" label Jul 13, 2024
@KaptainMicila KaptainMicila removed their assignment Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers requires more testing "I didn't manage to replicate this issue, so more science is required"
Projects
None yet
Development

No branches or pull requests

2 participants