Skip to content

Fixes #1326 - remove unused font awesome library#1327

Draft
trubach wants to merge 1 commit into
5.xfrom
issue-1326-remove-fontawesome
Draft

Fixes #1326 - remove unused font awesome library#1327
trubach wants to merge 1 commit into
5.xfrom
issue-1326-remove-fontawesome

Conversation

@trubach

@trubach trubach commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Closes #1326

Summary

Removes the bundled Font Awesome 4.7.0 library and the dead  illinois-icons.css  stylesheet from the theme. Font Awesome was loaded site-wide via the  global  library but only used for 3 decorative arrow glyphs, and  illinois-icons.css  was never matched by any markup (icons render through the Toolkit  <ilw-icon>  web component). This removes ~43 unnecessary files and trims the CSS shipped on every page.

Changes

•  illinois_framework_theme.libraries.yml  — Removed both  font-awesome.css  and  illinois-icons.css  from the  global  library's  css.theme  section.
•  css/node/node.news.css  & css/node/node.blog.css  — Replaced the 3 Font Awesome arrow glyphs with plain Unicode characters (rendered in the system font, no icon font required) and removed the now-unneeded  font-family: FontAwesome;  declarations:
•  \f054  (chevron-right) →  \203A  ( › ) on  .readmore::after  — both files
•  \f101  (angle-double-right) →  \BB  ( » ) on  .hvrbox-title::after  — news
• Deleted  fonts/font-awesome-4.7.0/  — Removed the entire directory (43 files): Font Awesome CSS/min/map, webfonts +  FontAwesome.otf , the  Illinois-Icons-Line.*  font files,  illinois-icons.css , the  less/  and  scss/  source dirs, and  HELP-US-OUT.txt .
•  GEMINI.md  — Updated to drop the Font Awesome references in the library description and External Dependencies sections.

What was intentionally kept

• The  $icon_map  array and  cta_icon  logic in  illinois_framework_theme.theme  — these translate stored icon keys into    icon names and are unrelated to the removed CSS.

Testing / verification

•  grep  confirms zero remaining references to  font-awesome ,  fontawesome , or  illinois-icons  across the theme, and no remaining  \f054 / \f101  glyphs.
• Static review confirms no template/JS uses  .fa / .fa-*  or  il-icon-*  classes.
• Note: Run  lando drush cr  after merge/deploy to flush the library and CSS-aggregation registry. Spot-check that "read more" links show  › , hover-box titles show  » , and that CTA, icon-row, and footer    icons still render.

Notes for reviewers / downstream

If any sub-theme or consumer references these stylesheets or the  fonts/font-awesome-4.7.0/  path directly, it will need the same update.


✅ Peer Review Checklist

Reviewers: Please verify the following before approving.

  • The pull request links to the issue it is addressing in the comment and in the "Development" section of the PR
  • There is a short summary that describes how the fix is implemented
  • Verifying work: Verify that the changes made are addressing the linked issue
  • Accessibility: Changes that might impact accessibility are reviewed to work with keyboard navigation and screen readers
  • Responsive: Layout is tested on desktop, tablet, and mobile screens
  • Config changes: Any configuration updates are tested and verified using the Distribution Update import
  • Security: Output is sanitized (using t(), Twig auto-escaping, etc.). No secrects included (API keys, etc.)
  • Cleanliness: All debug code (ksm(), dpm(), console.log) has been removed.
  • Comments: Complex logic is explained inline.
  • Documentation: Any relevant documentation has been updated (this can be a separate issue if needed)

📸 Screenshots / Video (Optional)

If this is a UI change, please attach a screenshot or a quick screen recording of the change in action.

@trubach trubach changed the title Fixes #1326 - remove unused fot awesome library Fixes #1326 - remove unused font awesome library Jun 30, 2026
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.

Remove unused Font Awesome library from the theme

1 participant