Skip to content

fix(frontend): optimize material symbols loading to fix slow icons and resolve pnpm validate audit issues#57

Merged
martingaldeca merged 2 commits into
mainfrom
fix-frontend-issues
Feb 21, 2026
Merged

fix(frontend): optimize material symbols loading to fix slow icons and resolve pnpm validate audit issues#57
martingaldeca merged 2 commits into
mainfrom
fix-frontend-issues

Conversation

@martingaldeca

Copy link
Copy Markdown
Member

Type of Change

  • 🚀 New feature (non-breaking change which adds functionality)
  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 🧹 Chore (technical task, refactoring, maintenance)

Description

Briefly describe the changes made in this Pull Request. Explain the solution and the reasoning.
Switched from manually importing material-symbols/outlined.css to loading from Google Fonts CDN to fix slow load times and eliminate FOUT (Flash of Unstyled Text) for icons in production. Added base utility class in globals.css to restore mappings.
Also updated dev dependencies in pnpm overrides to resolve multiple pnpm audit security vulnerabilities (minimatch, tar, fast-xml-parser, ajv) that were failing the CI validate hook.

Related Issue

Closes #

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

  • Unit Tests
  • Manual Testing

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

# Conflicts:
#	package.json
#	pnpm-lock.yaml
@martingaldeca martingaldeca merged commit 2cc3e0e into main Feb 21, 2026
2 checks passed
@martingaldeca martingaldeca deleted the fix-frontend-issues branch February 21, 2026 10:18
@martingaldeca martingaldeca mentioned this pull request Feb 21, 2026
18 tasks
martingaldeca added a commit that referenced this pull request Feb 21, 2026
…d resolve pnpm validate audit issues (#57)

## Type of Change

- [ ] 🚀 New feature (non-breaking change which adds functionality)
- [x] 🐛 Bug fix (non-breaking change which fixes an issue)
- [ ] 🧹 Chore (technical task, refactoring, maintenance)

## Description

Briefly describe the changes made in this Pull Request. Explain the
solution and the reasoning.
Switched from manually importing `material-symbols/outlined.css` to
loading from Google Fonts CDN to fix slow load times and eliminate FOUT
(Flash of Unstyled Text) for icons in production. Added base utility
class in globals.css to restore mappings.
Also updated dev dependencies in pnpm overrides to resolve multiple pnpm
audit security vulnerabilities (minimatch, tar, fast-xml-parser, ajv)
that were failing the CI validate hook.

## Related Issue

Closes #

## How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

- [ ] Unit Tests
- [x] Manual Testing

## Checklist

- [x] My code follows the project's style guidelines
- [x] I have performed a self-review of my code
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [x] New and existing unit tests pass locally with my changes

## Type of Change

- [ ] 🚀 New feature (non-breaking change which adds functionality)
- [ ] 🐛 Bug fix (non-breaking change which fixes an issue)
- [ ] 🧹 Chore (technical task, refactoring, maintenance)

## Description

Briefly describe the changes made in this Pull Request. Explain the
solution and the reasoning.

## Related Issue

Closes # (Link the issue here, e.g., #123)

## How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

- [ ] Unit Tests
- [ ] Manual Testing

## Checklist

- [ ] My code follows the project's style guidelines
- [ ] I have performed a self-review of my code
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes

## Screenshots (if applicable)
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.

1 participant