Closed tksheppard closed 9 months ago
Kudos, no new issues were introduced!
0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
@david-poindexter I know we discussed multiple PR's, but I noticed issue #358 already outlined the necessary changes to font usage and include updates.
It's still tossing the warnings that @dnndiva was speaking about earlier, but I'm not sure what to do about these. This seems like a new warning. I even tried making sure the page is using fontawesome and it still gives these warnings.
@david-poindexter I know we discussed multiple PR's, but I noticed issue #358 already outlined the necessary changes to font usage and include updates.
It's still tossing the warnings that @dnndiva was speaking about earlier, but I'm not sure what to do about these. This seems like a new warning. I even tried making sure the page is using fontawesome and it still gives these warnings.
@tksheppard it is totally fine for this to all be in one PR - thank you!
As it relates to the warning, this is for a woff
file. At this point, I see no reason why need to continue including woff
files. woff2
is supported now on all modern browsers. I will approve this PR and I'll create a separate issue for the drop of woff
support in nvQuickTheme.
Related to Issue
Fixes #357
Description
How Has This Been Tested?
Made updates, ran
gulp
. Compiled and checked on local environment to make sure no errors.Screenshots (if appropriate):
Types of changes
Checklist: