Dalamud icon indicating copy to clipboard operation
Dalamud copied to clipboard

Missing FontAwesome 5 Free Icons in IconFont

Open MalRD opened this issue 2 years ago • 2 comments

https://github.com/goatcorp/Dalamud/blob/a2305c7a1d39396076514df53f2451a037af7685/Dalamud/Interface/Internal/InterfaceManager.cs#L525

The IconFont currently only includes icons in Font Awesome 5 Free-Solid-900.otf. To include all of the free icons, Font Awesome 5 Free-Regular-400.otf and Font Awesome 5 Brands-Regular-400.otf should be included as well.

MalRD avatar Nov 30 '21 22:11 MalRD

we probably don't want to use the entire font atlas for just font awesome icons

Caraxi avatar Nov 30 '21 23:11 Caraxi

I put a commit in my fork that shows the change that need to be made even if no new font is added, as some of the unicode values are wrong in FontAwesomeIcon.cs. They were inadvertently changed during a code cleanup.

I was able to find a few icons that didn't match the latest FontAwesome values (and upon further inspection, didn't exist in 5.0.9), but these are Brand icons, so pointing to nothing doesn't cause any problems.

Excluding the Brand file makes sense to me, unless someone can describe a good use for it. There's a huge number of icons in that, most of which are useless in the context of addons. Regular is smaller, but upon further reflection, I'm not sure if there is any iconography that is exclusive to Regular. When someone mentioned icons were missing and scrolled through /xldata, I figured they meant the missing fonts, but it could just be the ~80 or so fonts that have the wrong unicode value in the FontAwesomeIcon.cs file.

MalRD avatar Dec 01 '21 16:12 MalRD