LearnNewsExamplesServices
Frontmatter
id9100
titlePerf: Remove unused Material Icons from Portal index.html
stateClosed
labels
enhancementaiperformance
assigneestobiu
createdAtFeb 11, 2026, 11:04 AM
updatedAtFeb 11, 2026, 11:21 AM
githubUrlhttps://github.com/neomjs/neo/issues/9100
authortobiu
commentsCount2
parentIssue8930
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtFeb 11, 2026, 11:21 AM

Perf: Remove unused Material Icons from Portal index.html

Closed v12.0.0 enhancementaiperformance
tobiu
tobiu commented on Feb 11, 2026, 11:04 AM

After verifying that the apps/portal codebase and its associated SCSS files do not contain any references to "material-icons", we can safely remove the Google Fonts link for Material Icons from apps/portal/index.html.

Goal: Remove the unused Material Icons dependency to improve page load performance.

Verification: grep searches across apps/portal and resources/scss confirmed zero usage.

tobiu added the enhancement label on Feb 11, 2026, 11:04 AM
tobiu added the ai label on Feb 11, 2026, 11:04 AM
tobiu added the performance label on Feb 11, 2026, 11:04 AM
tobiu added parent issue #8930 on Feb 11, 2026, 11:05 AM
tobiu
tobiu Feb 11, 2026, 11:12 AM

Input from Gemini 3 Pro:

✦ I incorrectly assumed Material Icons were unused because of a case-sensitive grep mismatch ("material-icons" vs "Material Icons"). The TreeList.scss component, shared by Portal and DevRank, explicitly sets font-family: 'Material Icons'.

Revised Plan:

  1. Refactor resources/scss/src/app/content/TreeList.scss to use FontAwesome instead of Material Icons.
    • Replace \e5cc (Chevron Right) with \f054 (FontAwesome fa-chevron-right).
    • Replace \e5cf (Chevron Down) with \f078 (FontAwesome fa-chevron-down).
    • Update font-family to "Font Awesome 6 Free".
    • Ensure font-weight: 900 is set (required for FA solid icons).
  2. Also fix resources/scss/src/apps/legit/Viewport.scss which has the same issue.

This will allow us to keep the removal of the Material Icons dependency from index.html.

tobiu assigned to @tobiu on Feb 11, 2026, 11:16 AM
tobiu
tobiu Feb 11, 2026, 11:21 AM

Input from Gemini 3 Pro:

✦ I have resolved this issue as part of the broader work in #9101 ("Fix: Standardize Font Awesome version usage in SCSS").

Resolution: The Material Icons dependency was removed from apps/portal/index.html as planned. The visual assets (Chevron icons in the TreeList) that were relying on it have been migrated to FontAwesome 7 Free, eliminating the need for the Material Icons font file.

This ensures both the performance gain (fewer requests) and visual consistency.

tobiu closed this issue on Feb 11, 2026, 11:21 AM