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

Disabled #ifdef section is orange #5

Open
2 tasks done
Wuerfel21 opened this issue Apr 10, 2024 · 5 comments
Open
2 tasks done

Disabled #ifdef section is orange #5

Wuerfel21 opened this issue Apr 10, 2024 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@Wuerfel21
Copy link

and have you...

  • I updated to the latest spin2 version available
  • I have restarted VSCode and the problem still exists

Version of Spin2 Language Server VScode Extension with the issue

the new one

Is this a regression?

Yes. This used to work, but now it doesn't.

Hardware, Operating System, Operating System version, VSCode version

[insert the funny here]

Description of problem

An attempt seems to have been made at greying out the disabled #ifdef sections. Unfortunately, it gives everythng the orange color normally used for strings:
image

This working correctly in the C++ extension:
image

Additional information

No response

@Wuerfel21 Wuerfel21 added the bug Something isn't working label Apr 10, 2024
@Wuerfel21
Copy link
Author

(there don't appear to be any special scopes created by the C++ extension, so it must be doing something else to lower the opacity)

@ironsheep
Copy link
Owner

wow... my apologies! ;-)

Ok, a couple of things. My bad for not thinking about other themes and how they'd interpret things.

Please help me with two things:

  1. Tell me exactly which C++ extension you are using, and I'll go wander the source code to see how they are doing it.
  2. tell me exactly which theme you are using so I can see how I'm triggering the orange.

At this point, I don't know if I can runtime affect colors so in my own themes I was planning on a 2nd dimmer full-color palette which would be selected by the attribute that I'm currently using to select the single grey color (well in my themes it's grey ;-)

Maybe the C++ code investigation will show that there is another possible way to do this.

@ironsheep ironsheep self-assigned this Apr 10, 2024
@Wuerfel21
Copy link
Author

@Wuerfel21
Copy link
Author

Also, something I notice is that sometimes large sections of the file turn orange for no apparent reason. It seems to never clear which lines should be disabled when editing the file. But sometimes it also happens without having made large edits

@ironsheep
Copy link
Owner

Sigh.... good to know...

I found problems with how they were doing the coloring... I'm still investigating if it's possible to use their technique.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants