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 t-class directive? #1546

Open
ged-odoo opened this issue Oct 22, 2023 · 0 comments
Open

add t-class directive? #1546

ged-odoo opened this issue Oct 22, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@ged-odoo
Copy link
Contributor

Being able to specify a css class for the root element of a child component is very useful. We removed it going from owl 1 to owl 2 because it could not really be implemented properly in a generic way, and also, because it couples the implementation of the component with its use. However, i am starting to think that all these class, root or className props are not really much better. What if we allow adding classes to the root element, and it throws error if there is a branching?

<div>
  <Child t-class="hello"/>
</div>

Even if we decide that the given class is purely static, it could be useful.

@ged-odoo ged-odoo changed the title add t-class directive add t-class directive? Oct 22, 2023
@sdegueldre sdegueldre added the enhancement New feature or request label 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
Projects
None yet
Development

No branches or pull requests

2 participants