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

Can't set class= attribute #62

Open
amk221 opened this issue Apr 10, 2019 · 13 comments
Open

Can't set class= attribute #62

amk221 opened this issue Apr 10, 2019 · 13 comments

Comments

@amk221
Copy link

amk221 commented Apr 10, 2019

I'm not sure what combination of dependencies causes this, but a simple example of:

<MyComponent class="foo" />

is not rendering the class attribute.... changing to @class fixes it, but that's wrong?

Demo repo with failing test:

https://github.com/amk221/-ember-angle-brackets-in-addon/blob/master/tests/integration/components/my-component-test.js#L13

@rwjblue
Copy link
Member

rwjblue commented Apr 10, 2019

Looks like you are hitting the issue reported in emberjs/ember.js#17533. This should have been fixed in [email protected] (by emberjs/ember.js#17870), but it looks like you are already using 3.9.1 and therefore shouldn't have this issue... 🤔

@chancancode - Anything obvious I missed?

@rwjblue
Copy link
Member

rwjblue commented Apr 10, 2019

@amk221 - I've kicked off #63 to see if the test we added shows an issue under 3.9...

@amk221
Copy link
Author

amk221 commented May 10, 2019

👋🏻Any update on this?

@amk221
Copy link
Author

amk221 commented Aug 20, 2019

Hi again, any idea what I can do to help with this?

@rwjblue
Copy link
Member

rwjblue commented Aug 20, 2019

Maybe send in a failing test showing input / output expectations? I was fairly sure that this was fixed...

@amk221
Copy link
Author

amk221 commented Aug 20, 2019

I did do a failing test in the example repo, but I will try to add one directly to ember-angle-bracket-invocation-polyfill

@amk221
Copy link
Author

amk221 commented Aug 20, 2019

I've done a bit of digging...

  • If I install ember-font-awesome into my failing test demo repo, then it passes.

  • Similarly, if I add my failing test from the demo repo to ember-angle-bracket-invocation-polyfill, and uninstall ember-font-awesome, then that test fails with ember try:one ember-lts-3.8

I realise this is a bit odd and maybe not helpful.

I hope it’s not something stupid. I’d hate it to be a red herring.

@rwjblue
Copy link
Member

rwjblue commented Aug 21, 2019

Yikes. That is indeed odd.

@amk221
Copy link
Author

amk221 commented Aug 22, 2019

Here is a build of ember-angle-bracket-invocation-polyfill without ember-font-awesome

Most builds pass, apart from ember-3.4 and ember-3.8

@amk221
Copy link
Author

amk221 commented Oct 7, 2019

👋 Being able to set class= as an attribute is a very common requirement...

I'm starting to get a backlog of test failures on travis where this polyfill isn't working. Making it impossible to release new versions of my addons with backwards compatibility.

I'm surprised nobody else is experiencing this

Copy link
Member

rwjblue commented Oct 7, 2019

I'm sorry, I haven't had enough time lately to be able to dig into your demo. I will try to put this back on my queue to dig into, but I'm not sure what that means timeline wise...

@amk221
Copy link
Author

amk221 commented Oct 7, 2019

That's ok.

In the meantime, I will try to factor out font-awesome from ember-angle-bracket-invocation-polyfill's test suite

@Panman82
Copy link

FWIW, I think I'm running into this too. It seems passing class="" into a splattributes isn't working below 3.9. Any chance this polyfill can account for the issue? Or is that something that would need to be backported in ember (which is not likely to happen)?
https://travis-ci.org/froala/ember-froala-editor/builds/624378772

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

No branches or pull requests

3 participants