themeteorchef / bert

A client side, multi-style alerts system for Meteor.
185 stars 27 forks source link

Request clarification on how to install/use fontawesome under 2.2.0? #55

Open philipmjohnson opened 5 years ago

philipmjohnson commented 5 years ago

I've been using Bert Alerts successfully, but noticed that since updating to 2.2.0, I no longer get the icon with an alert:

Screen Shot 2019-04-10 at 10 48 31 AM

The associated invocation of Bert Alert is here:

https://github.com/openpowerquality/opq/blob/master/view/app/imports/ui/components/AboutMe.jsx#L134

I installed the fontawesome library with:

meteor npm install --save @fortawesome/fontawesome-free
npm WARN leaflet.markercluster@1.3.0 requires a peer of leaflet@~1.0.3 but none is installed. You must install peer dependencies yourself.
npm WARN react-meteor-data@0.2.11 requires a peer of react@^0.14.0 || ^15.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN uniforms@1.24.4 requires a peer of graphql@>=0.8.2 <1.0.0 but none is installed. You must install peer dependencies yourself.

+ @fortawesome/fontawesome-free@5.8.1
added 1 package from 6 contributors and audited 2304 packages in 6.591s
found 1 moderate severity vulnerability
  run `npm audit fix` to fix them, or `npm audit` for details

Your thoughts on how I can diagnose and fix this? Thanks!

andsnw commented 5 years ago

Encountering the same issue!

RealHandy commented 5 years ago

I'm also having this issue.

andsnw commented 5 years ago

A hacky solution using old dependency:

  1. Install the font-awesome dependency: meteor add fortawesome:fontawesome
  2. Clone the package locally into /packages, replace 'fas' with 'fa' on lines 20-26 in bert.js: image
  3. Remove package and re-install locally: meteor remove themeteorchef:bert; meteor add themeteorchef:bert

Success, icons should now work again.

zachspangler commented 4 years ago

For me, the issue was as simple as adding the fontawesome v5 stylesheet link in the head.html of the project.