Skip to content

Conversation

@notsure
Copy link
Collaborator

@notsure notsure commented Jul 31, 2025

Pull request

  • Added gzip compression for production build
  • add a static name for the manifest to avoid vite default (.vite/manifest.json)
  • introduced first attempt of manual chunks to split the code into some shared parts.

Ticket

Browser testing

  • npm run dev > works like before
  • npm run build:profile > see different chunks
  • npm run build > check the dist folder - you will see gz files and a manifest.json

Checklist

  • I merged the current main branch (before testing)
  • Added JSDoc and styleguide demo
  • Tested all links in project relevant browsers
  • Tested all links in different screen sizes
  • Did run automated tests and linters
  • Did assign ticket
  • Double checked target branch

Review/Test checklist

  • Did review code and documentation
  • Tested all links in project relevant browsers
  • Tested all links in different screen sizes
  • Did check accessibility (Wave, only errors)
  • Re-assign ticket to developer

@notsure notsure added the enhancement New feature or request label Jul 31, 2025
@notsure notsure self-assigned this Jul 31, 2025
@notsure notsure requested a review from mob8607 July 31, 2025 06:54
Copy link
Collaborator

@mob8607 mob8607 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but needs to be tested in a project I guess.

@notsure notsure merged commit e2356a8 into main Jul 31, 2025
1 check passed
@notsure notsure deleted the feature/performance-fixes branch July 31, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants