-
Notifications
You must be signed in to change notification settings - Fork 49
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
Is there anyway to exclude the js file from the bundle? #52
Comments
Are you talking about the |
Yeah, that one :)
…On Wed, Jun 27, 2018, 4:18 PM Rob Madole ***@***.***> wrote:
Are you talking about the fontawesome.js? The 7.87 KB entry?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#52 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMJamHT6b3PaMy2fiZ6FDMPe6TZJNgqks5uA-ihgaJpZM4U334r>
.
|
Are you pre-rendering the icons @NullVoxPopuli ? I'm a little confused as to the question here. Aren't you using |
Yeah, ember-fontawesome https://gitlab.com/NullVoxPopuli/emberclear/blob/master/packages/frontend/package.json#L39 I'm both prerendering (prember) and not prerendering (non-prember pages), but I was hoping that the fa icon helper could be replaced by i tags at build time, ya know? Just so there is no runtime evaluation / rendering of the helper (so, the Js then wouldn't be required :) ) |
@NullVoxPopuli it sounds doable to have a configuration option to only allow build time rendering and never include the component or any of the backing JS from fontawesome. I'm not volunteering mind you :) |
any movement on this? |
@burritoIand not yet. If you're interested in getting involved and helping jump into #e-fontawesome in discord and let's chat. |
I'm looking at broccoli-concat-analyser, and see:
it's in tiny text on the right. what's the js for? can I remove it somehow?
The text was updated successfully, but these errors were encountered: