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

Doesn’t work with angle-bracket components #61

Closed
andrewbranch opened this issue May 21, 2015 · 13 comments
Closed

Doesn’t work with angle-bracket components #61

andrewbranch opened this issue May 21, 2015 · 13 comments

Comments

@andrewbranch
Copy link

{{my-component}} works like a charm but <my-component /> doesn’t get the hashed identifier class added.

@andrewbranch
Copy link
Author

I guess we’ll come back to this when angle-bracket components re-land in a beta. I might have time to take a look at it then.

@oligriffiths
Copy link

@andrewbranch I'm using angle brackets and it works fine. It doesn't work however when building for production.

@andrewbranch
Copy link
Author

This is not a thing anymore. Support for angle brackets was removed shortly after I filed.

@eguitarz
Copy link

@andrewbranch @ebryn Angle brackets is landed in Ember 3.3. Should we reopen this issue? RFC PR is here: emberjs/rfcs#311

@tleperou
Copy link

+1 sounds good to support angle brackets!

@webark webark reopened this Sep 15, 2018
@webark
Copy link
Owner

webark commented Sep 15, 2018

does anyone have any clues as to why it’s not working? or a simple repro app..?

@tleperou
Copy link

tleperou commented Sep 16, 2018

#57 relates the issue:

The problem is that angle bracket components hit ComponentLookup.componentFor() instead of 'ComponentLookup.lookupFactory()'. So, we just need to override componentFor as well. I'll add a pull request.

Unfortunately, I don't find any public documentation about componentFor.

Any clue?

@webark
Copy link
Owner

webark commented Sep 16, 2018

@jasonworden
Copy link

I want to help. Anyone have an idea where to get started here?

@tleperou
Copy link

tleperou commented Mar 1, 2019

Same one here.

We have to check out the ember-octane and particularly the @glimmer/component to ensure the compatibility. Unfortunately, I am not fully familiar - yet - with those packages.

Feel free to share every clue you have, I would help you.

@jasonworden
Copy link

@tleperou Actually, just tried, and it looks like this is already working with <AngleBracket /> components! I initially just took it for granted that it wouldn't work, but I'm not seeing any issues, and the styles applied fine!

Note: I'm running Ember 3.4.7.

@webark
Copy link
Owner

webark commented Mar 1, 2019

closing then on @jasonworden comments. If it ends up being not the case, feel free to reopen @tleperou

@BryanCrotaz
Copy link

How did you access the style namespace? I've tried:

<div class="{{@styleNamespace}}">

and

<div class="{{this.styleNamespace}}">

Neither of them produce a class attribute

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants