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 adding t-ref on text nodes #1470

Open
ged-odoo opened this issue Jul 3, 2023 · 1 comment
Open

allow adding t-ref on text nodes #1470

ged-odoo opened this issue Jul 3, 2023 · 1 comment
Labels
enhancement New feature or request need info More information is needed to proceed

Comments

@ged-odoo
Copy link
Contributor

ged-odoo commented Jul 3, 2023

something like this:

<t t-ref="someRef">text</t>
@sdegueldre
Copy link
Contributor

sdegueldre commented Jul 5, 2023

Not necessarily against it but curious about the use case. Also this may not be super straight forward as this probably also means we want to support it on <t t-esc="stuff"/> nodes which is not the same type of BDom node, and it can generate potentially difficult to understand errors when the node has more than one child node, including comment nodes:

<t t-ref="brokenRef">
  some text
  <!-- comment explaining something -->
  more text
</t>

@sdegueldre sdegueldre added enhancement New feature or request need info More information is needed to proceed labels Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request need info More information is needed to proceed
Projects
None yet
Development

No branches or pull requests

2 participants