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

PHP highlighting not working #21

Closed
atymic opened this issue Oct 20, 2019 · 8 comments
Closed

PHP highlighting not working #21

atymic opened this issue Oct 20, 2019 · 8 comments
Labels
bug Something isn't working

Comments

@atymic
Copy link

atymic commented Oct 20, 2019

Hey 👋

It seems that PHP code blocks without a <?php opening tag don't get colorised correctly.

I've tested and if you add a <?php tag, everything starts working correctly.

I guess this is probably an upstream issue?

@atymic
Copy link
Author

atymic commented Oct 20, 2019

So i've tested in VScode and it works correctly. I found these issues related to it:

microsoft/vscode#14166
microsoft/vscode#16502

So i'm not sure why it doesn't work here. I think maybe because the actual grammar file is not the same as the one used by vscode?

@octref octref added the bug Something isn't working label Oct 22, 2019
@octref
Copy link
Collaborator

octref commented Aug 21, 2020

I think the behavior is consistent with VS Code:

image

image

@D-Marc1
Copy link

D-Marc1 commented Aug 21, 2020

@octref It works if you do it in markdown on VSCode though.

Screen Shot 2020-08-21 at 12 04 04 PM

@octref
Copy link
Collaborator

octref commented Aug 22, 2020

I can only allow 1 PHP grammar. If I change it, people who has written <?php in their PHP code would complain to me why it's not syntax highlighted correctly.

@D-Marc1
Copy link

D-Marc1 commented Aug 22, 2020

Unless I'm missing something, wouldn't it be more versatile to use the markdown version of PHP as the 1 PHP grammar? Appending <?php still works for me, so it should still be supported for people who are using it.

Screen Shot 2020-08-22 at 7 28 15 AM

@octref
Copy link
Collaborator

octref commented Aug 26, 2020

Seems working in 0.2.0:

image

@octref octref reopened this Aug 26, 2020
@octref octref closed this as completed in 3a7e618 Aug 26, 2020
@D-Marc1
Copy link

D-Marc1 commented Aug 28, 2020

You're the man, @octref! Big thanks for all of your hard work in OSS!

@atymic
Copy link
Author

atymic commented Aug 28, 2020

Awesome job, thanks @octref 👍

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

3 participants