excalidraw/excalidraw

move to `iTXt` png chunks

Aperta

#9269 aperta il 13 mar 2025

Vedi su GitHubGuida contributor
 (5 commenti) (1 reazione) (1 assegnatario)TypeScript (14.984 fork)batch import
good first issue

Metriche repository

Star
 (130.414 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

When encoding scene data to PNGs (see https://github.com/excalidraw/excalidraw/blob/master/packages/excalidraw/data/image.ts)

turns out tEXt chunks support latin1 only, but we're encoding the data as utf8. Browsers don't care, but it may throw off other PNG decoders.

Instead, we should use iTXt chunks which support utf8.

https://www.npmjs.com/package/png-itxt png-chunk-itxt

We still need to keep tEXt support for backwards compat.

Guida contributor

Direzione di ricerca
Ispeziona la codifica tEXt corrente in packages/excalidraw/data/image.ts, comprendi come vengono codificati i dati della scena, quindi utilizza la libreria png chunk itxt per passare a chunk iTXt mantenendo la compatibilità con tEXt.
Tech stack
typescript
Dominio
data
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75