sindresorhus/eslint-plugin-unicorn

Rule proposal: `no-canvas-to-image`

Chiusa

#1271 aperta il 13 mag 2021

Vedi su GitHubGuida contributor
 (1 commento) (3 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

Many people don't know CanvasRenderingContext2D.drawImage() accept HTMLCanvasElement as CanvasImageSource

Fail

const image = await loadImage(canvas1.toDataURL());
context2.drawImage(image);

const imageData = context1.getImageData();
context2.putImageData(imageData);

Pass

context2.drawImage(canvas1)

Guida contributor

Direzione di ricerca
Implementa una nuova regola ESLint 'no canvas to image' che rilevi conversioni inefficienti da canvas a immagine. Studia le regole esistenti nel repository eslint plugin unicorn (ad esempio, nella cartella 'rules') per comprenderne il modello. Crea un file di regola che utilizzi selettori AST per trovare chiamate come `canvas.toDataURL()` utilizzate in `loadImage` o `new Image()`. Scrivi i test corrispondenti nella cartella 'test'. La regola dovrebbe suggerire di utilizzare direttamente `drawImage(canvas)`. Basa la tua implementazione sulla struttura di altre regole che controllano l'uso improprio delle API.
Tech stack
javascript
Dominio
frontendperformance
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
understanding of ESLint plugin developmentfamiliarity with AST manipulation
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65