parallax/jsPDF

Context2D does not implement clip() correctly

Open

#3 408 ouverte le 9 avr. 2022

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (28 280 stars) (4 596 forks)batch import
Feature Requestdifficulty:mediumhelp wanted

Description

clip() is supposed to take an argument to specify the clipping rule. Context2D drops this argument on the floor and implements the default clipping rule, I guess.

It can also be given two arguments where one argument is the path. This is also not implemented.

https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/clip

Guide contributeur

Stack technique
javascript
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
JavaScriptCanvas APIjsPDF internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50
Direction de recherche
Explore the Context2D implementation in jsPDF, likely in src/context2d.js or similar. The current clip() method should be located and compared with the MDN specification. The fix needs to handle the optional `fillRule` argument and the two argument variant where the first argument is a path. Look for how other methods like `fill()` or `stroke()` handle similar arguments. Review any existing tests for clip() in the test directory.