parallax/jsPDF

Context2D does not implement clip() correctly

Open

#3.408 geöffnet am 9. Apr. 2022

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (28.280 Stars) (4.596 Forks)batch import
Feature Requestdifficulty:mediumhelp wanted

Beschreibung

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

Contributor Guide

Tech Stack
javascript
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
JavaScriptCanvas APIjsPDF internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50
Research-Richtung
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.