search

bajrangCoder / zed-laravel-blade

Larvel Blade templating language support for Zed
12 stars 0 forks source link

Php injection seems broken #5

Open AlexandreBonaventure opened 2 months ago

AlexandreBonaventure commented 2 months ago

Hi, I tried this extension but it seems that the php injection is broken. When switching the file to Blade, I get the syntax highlights for html directives, but the php code block above is broken: no highlights, no intellisense. It would be so nice if this is working, I can help but wanted to first see with you if that's expected. Thanks

Tested on Zed v0.149.5

Capture d’écran, le 2024-08-23 à 12 08 20 Capture d’écran, le 2024-08-23 à 12 08 52
bajrangCoder commented 2 months ago

It was working fine before (except for the IntelliSense), but it seems the recent Zed updates caused it to break. I'm currently working on a fix.

Thanks for reporting this!

bajrangCoder commented 2 months ago

after fixes: Screenshot_20240825_092436

still inline php highlighting isn't working

AgungBahtiarr commented 2 months ago

after fixes: Screenshot_20240825_092436

still inline php highlighting isn't working

is the autocomplete for the .blade.php file still not working?

bajrangCoder commented 2 months ago

after fixes: Screenshot_20240825_092436 still inline php highlighting isn't working

is the autocomplete for the .blade.php file still not working?

Yes it's not working but it's defined https://github.com/bajrangCoder/zed-laravel-blade/blob/7aba92bc95c50890ba2d4eb1e2f5aeb2e795fce2/languages/blade/config.toml#L20