Skip to content

Conversation

94noni
Copy link
Contributor

@94noni 94noni commented Aug 17, 2024

comment "inside php code" render in wrong color
Capture d’écran 2024-08-17 à 17 30 57

@carsonbot carsonbot added this to the 5.4 milestone Aug 17, 2024
@OskarStark
Copy link
Contributor

friendly ping @wouterj

@javiereguiluz
Copy link
Member

Thanks Antoine!

Let's merge this one as a temporary fix ... and, in addition, I created symfony-tools/docs-builder#180 in case we can fix this upstream in the code highlighter used in Symfony Docs.

@javiereguiluz javiereguiluz merged commit 73aa094 into symfony:5.4 Aug 26, 2024
@94noni 94noni deleted the patch-12 branch August 26, 2024 12:02
javiereguiluz added a commit to symfony-tools/docs-builder that referenced this pull request Dec 31, 2024
This PR was merged into the main branch.

Discussion
----------

Highlight nested PHP comments

Related to symfony/symfony-docs#20121

This PR adds a failing test that shows the issue. This is the output change that would make test pass:

```diff
-                ), [ // wait
-                <span class="hljs-number">5</span>
-                seconds before processing
+                ), [
+                <span class="hljs-comment">// wait 5 seconds before processing</span>
```

Commits
-------

af7ee47 Allow one line comments in argument list
8f09af6 Highlight nested PHP comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants