symfony / ux

Symfony UX initiative: a JavaScript ecosystem for Symfony
https://ux.symfony.com/
MIT License
859 stars 314 forks source link

[TwigComponent] change `cva` to `html_cva` #2291

Closed seb-jean closed 1 month ago

seb-jean commented 1 month ago

cva twig function is deprecated. Is it worth changing the documentation to use the html_cva function?

smnandre commented 1 month ago

There is already a note about it, do you think we should make it more visible ?

Capture d’écran 2024-10-23 à 08 54 33

Renaming: i'm not sure, as keeping the current naming will be more usefull for people searching in search engine, using previous Twig Component versions, etc... at least until version 3.0 :)

Then... i doubt we maintain the documentation here as it is now a official Twig package and has it's own documentation 🤷‍♂️

seb-jean commented 1 month ago

Yes, it doesn't make sense to change the documentation indeed.