Skip to content

Conversation

@userquin
Copy link
Member

@userquin userquin commented May 26, 2025

Description

Before merging check #365 , maybe you also want a web worker.

This PR using ESM version of svg-packer with Vite, replacing also jszip with client-zip, including:

  • update vite to v6.3.5
  • update vite-plugin-pwa to v1.0.0

We need to review electron, no idea how to test it and do some check about generated zips, all seems to be fine but exporting full collections getting huge differences, for example exporting mdi, current version ~3MB while the new version ~14MB.

#363 moved here (branch here)

supersedes #363

Linked Issues

resolves #362

Additional context

We should add a web worker or reuse the service worker to generate the zip files.

The build result with 2MB less of js code 😍 :

image

@netlify
Copy link

netlify bot commented May 26, 2025

Deploy Preview for icones ready!

Name Link
🔨 Latest commit 3124bb8
🔍 Latest deploy log https://app.netlify.com/projects/icones/deploys/68369cf38c86030008f7366d
😎 Deploy Preview https://deploy-preview-364--icones.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@antfu antfu merged commit 5e8ea58 into main May 28, 2025
6 of 7 checks passed
@antfu antfu deleted the userquin/feat-use-esm-svg-packer branch May 28, 2025 05:26
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.

use client-zip instead jszip

3 participants