Skip to content

Remove Mustache from attribute values #13

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

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Remove Mustache from attribute values #13

merged 1 commit into from
Nov 21, 2024

Conversation

deathaxe
Copy link
Member

Mustache text interpolation

  1. is not supported within tag attribute values see: https://vuejs.org/guide/essentials/template-syntax.html#attribute-bindings

  2. supports expressions only see: https://vuejs.org/guide/essentials/template-syntax.html#expressions-only

    Note, JavaScript provides an expression context, but embedding it causes meta.interpolation scope to be cleared, if mustache content is not a valid expression. Hence keeping embed: scope:source.js for now.

    A possible alternative context expressions requires ST4180+.

Mustache text interpolation

1. is not supported within tag attribute values
   see: https://vuejs.org/guide/essentials/template-syntax.html#attribute-bindings

2. supports expressions only
   see: https://vuejs.org/guide/essentials/template-syntax.html#expressions-only

   Note, JavaScript provides an `expression` context, but embedding it causes
   `meta.interpolation` scope to be cleared, if mustache content is not a valid
   expression. Hence keeping `embed: scope:source.js` for now.

   A possible alternative context `expressions` requires ST4180+.
@deathaxe deathaxe merged commit 6f3d57b into main Nov 21, 2024
14 checks passed
@deathaxe deathaxe deleted the fix/mustache branch November 21, 2024 09:53
@FichteFoll FichteFoll added this to the 5.0.0 milestone Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants