-
Notifications
You must be signed in to change notification settings - Fork 33
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
Comments
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? |
👋🏻Any update on this? |
Hi again, any idea what I can do to help with this? |
Maybe send in a failing test showing input / output expectations? I was fairly sure that this was fixed... |
I did do a failing test in the example repo, but I will try to add one directly to |
I've done a bit of digging...
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. |
Yikes. That is indeed odd. |
Here is a build of Most builds pass, apart from ember-3.4 and ember-3.8 |
👋 Being able to set 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 |
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... |
That's ok. In the meantime, I will try to factor out font-awesome from ember-angle-bracket-invocation-polyfill's test suite |
FWIW, I think I'm running into this too. It seems passing |
I'm not sure what combination of dependencies causes this, but a simple example of:
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
The text was updated successfully, but these errors were encountered: