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

Feature/font awesome 5 #239

Closed
wants to merge 3 commits into from
Closed

Conversation

basz
Copy link

@basz basz commented Feb 6, 2019

attempt at #206

@basz
Copy link
Author

basz commented Mar 29, 2019

Ping! Is this project dead?

@donaldwasserman
Copy link

donaldwasserman commented Apr 12, 2019

@basz I'd also like to know that. I'd be happy to maintain this with some enhancements. Maybe this is something that we can add to the ember-adopted addons project?

I'm donaldwasserman on discord if you'd want to chat more about it.

I use this in work projects and am currently maintaining two separate ways of dealing with this same problem and would like to not to do that.

@basz
Copy link
Author

basz commented Apr 12, 2019

Guess that’s up to @ynnoj (ping)

He seems active enough but probably doesn’t have the time currently to maintain this repo.

@donaldwasserman
Copy link

@ynnoj This is a really great addon, but the issues are really piling up. I'd really like to help maintain this if you don't have enough bandwidth!

@ynnoj
Copy link
Contributor

ynnoj commented Jul 23, 2019

@donaldwasserman Hey! Would love to get you involved with steering the project and working on some ideas I've had.

How do we do this?

@NullVoxPopuli
Copy link

NullVoxPopuli commented Sep 7, 2019

This works for me!

Just had to add:

  ENV['ember-cli-notifications'] = {
    icons: 'fa-5',
  };

to my config/environment file

@juggy
Copy link

juggy commented Sep 27, 2019

This does not work on 3.12+. Removing the tagName: '' line makes it work. with no apparent UI problems.

@NullVoxPopuli
Copy link

@juggy, I'm using this branch on 3.13+ #246

@mansona
Copy link
Owner

mansona commented Nov 12, 2019

Hey folks 👋 just an update on this PR, we have wanted to make this a dependency-free addon for quite a while so we have removed Font Awesome as a necessary dependency and included SVGs for the icons instead. You can take a look at the upcoming release here #251 (comment)

I am really interested in knowing what you all think about this change

@mansona mansona closed this Nov 12, 2019
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

Successfully merging this pull request may close these issues.

6 participants