-
Notifications
You must be signed in to change notification settings - Fork 25
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
Loses class and other passed attributes #8
Comments
Ah the unstable world of functional Vue components! I haven't yet found a way to inherit attributes/classes/etc. between functional hocs. I'd love it anyone could come up with a solution. If you try the same with non-functional components it should work, so it depends whether you'd consider that as a workaround. Luckily, functional components in Vue aren't quite as essential as in React. |
@jackmellis Wow good to know. I'll think on a fix too. Should I leave it opened? |
Yeah I'd like to keep this issue open for now, HOCs and Functional components feel like they belong together (thanks React) so if anyone can put forward a solution it'd be greatly appreciated! |
Hope I find the time to take a look. At first sight, the |
same here, pretty headachy. |
what's the work around that does not involve adding new param into the wrapped component, just for appending class name and other cust params with values? |
just realize this has been A WHOLE YEAR already lol wow.... boy.... |
Yep, no progress I'm afraid. The workaround mentioned above is that if you use |
@LinusBorg ^^^ any hope? blockers? |
Sorry but I haven't really been working on this ever since, and won't have time to get back into it. |
saw your discussion there in vue main repo, thought helpful for anyone else jump in here, as its top result in google search... |
I've noticed that
createHOC
loosesdata.attributes
in produced components which leads to loosing class and other attributes in actual DOM.E.g.
When using in template like
Does not include class
green-text
But when defined like:
It does.
The text was updated successfully, but these errors were encountered: