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

Allow insert tags in link titles #6720

Merged
merged 1 commit into from Jan 22, 2024
Merged

Conversation

aschempp
Copy link
Member

I often use {{link_url::42}} for the page link, and then use {{link_title::42}} for the link title. That does not seem to work anymore with the Twig templates.

Not sure if that should also work on the link title (the HTML attribute) and how that would be accomplished in attrs...

@aschempp aschempp added the bug label Jan 16, 2024
@aschempp aschempp added this to the 5.2 milestone Jan 16, 2024
@aschempp aschempp requested a review from m-vo January 16, 2024 13:17
@aschempp aschempp self-assigned this Jan 16, 2024
Copy link
Member

@leofeyer leofeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would change the link text, not the link title. But I think that is what you intended.

@aschempp
Copy link
Member Author

yes, what I meant was I'm not sure if it should be allowed in the title as well, and how that would be done in Twig. But this sure is correct anyway.

Copy link
Member

@m-vo m-vo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 for replacing insert tags but not trusting the output here.

@leofeyer leofeyer changed the title Allow insert tags in link title Allow insert tags in link titles Jan 22, 2024
@leofeyer leofeyer merged commit 678e167 into contao:5.2 Jan 22, 2024
17 checks passed
@leofeyer
Copy link
Member

Thank you @aschempp.

@aschempp aschempp deleted the fix/link-title branch January 26, 2024 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants