Hello there :) Hopefully you don't begin to dread seeing my name! I have a couple bug reports and one suggestion:

The red-highlighted brackets (a recent change?) are helpful, but two things:
1: They don't work right with php tags in-between:

<?php

if (true) {
  ?>
  <p>Some HTML</p>
  <?php
}
?>

The above snippet incorrectly highlights the { and } in red as though it were an error, but it's not(?)

2: It's common for php files to begin with <?php but not have an ending ?>; this helps avoid whitespace issues, and indeed is enshrined in various coding standards (example). Therefore, the red-highlighted < for a PHP open tag without a closing tag suggests a problem that doesn't exist (closing tags are optional). It'd be nice to fix this, or at least make it configurable.

3: Found another "runaway indent":

<?php if (true): ?>
    <!-- 
                                        <?php echo "going right!"; ?>
    -->
<?php endif; ?>

    KrisReeves Hi Kris, thanks for reporting the issue, we appreciate it!

    1, 2, is working just fine for me. Isn't there another PHP extension in your VSCode?

    Edit: maybe you have incorrect file association in vscode, and .php files are treated as HTML language? Please check your status bar:

      About issue #2 - I also noticed that problem several days ago, but now it's disappeared, maybe fixed in recent updates.

        Issue 3: Thanks, will be fixed!

          1/2 - Seems to have been fixed by a VS Code update - red herring, sorry!
          I've been on the pre-release version of the plugin due to the other reports/fixes, and noticed it after an update, but included it here since it had been hanging around a while.

          BTW we bought the paid version almost immediately solely because of your responsiveness here, thanks for being awesome :)

          Thank you so much Kris! We really appreciate your support and providing us with constant feedback!

            KrisReeves thank you Kris! Your support makes it possible for us to keep working on the extension 24/7!

              The fix for 3 will be available in the pre-release.

              Since also 1,2 are resolved as well, I'm closing this one.

              Thank you!

                Write a Reply...