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

Add support for inline Twig comments #184

Merged

Conversation

javiereguiluz
Copy link
Collaborator

Twig added inline comments (twigphp/Twig#4349) so I tried to add support for them here. But, it's not working. Anyone knows if this could be easy to fix or if it's too complex to do it? Thanks!

@javiereguiluz javiereguiluz added the help wanted Extra attention is needed label Sep 27, 2024
@wouterj wouterj self-assigned this Sep 28, 2024
@wouterj
Copy link
Contributor

wouterj commented Sep 29, 2024

We are green now :)

@wouterj wouterj removed the help wanted Extra attention is needed label Sep 29, 2024
@javiereguiluz
Copy link
Collaborator Author

Wouter, you are amazing! 🙇 Thanks a lot for fixing this.

@javiereguiluz javiereguiluz merged commit 29ddf17 into symfony-tools:main Sep 30, 2024
2 checks passed
@javiereguiluz javiereguiluz deleted the twig_inline_comments branch September 30, 2024 06:10
@javiereguiluz
Copy link
Collaborator Author

After merging and releasing this, I rebuilt the Symfony Docs and we can see that this is already working: https://symfony.com/bundles/ux-toggle-password/current/index.html#usage-without-symfony-forms

image

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants