PDF import fixes

Charlotte Curtisrequested to merge
cfcurtis/inkscape:pdf-clip-fix into master

This fixes some bugs introduced in MR 6766, namely:

  • The page affine was incorrectly assumed to be applied separately and not needed to compare paths
  • Rejecting clipping paths broke the behaviour of nested clipping and masking groups
  • Clipping path containment did not consider the defined winding rule

In addition, _shouldClip has been changed to _checkClip, returning a Geom::PathVector instead of a boolean. This removes some redundant processing of the clipping path, but has the side effect of changing the transforms that are written to file (as the page and clipping path transforms are now baked into the path). I've updated the test cases to reflect this change, but please let me know if this is undesired behaviour.

Merge request reports