Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
  • extensions extensions
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 76
    • Issues 76
    • List
    • Boards
    • Service Desk
    • Milestones
    • Requirements
  • Merge requests 13
    • Merge requests 13
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • InkscapeInkscape
  • extensionsextensions
  • Merge requests
  • !427

fix reverse of move command

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Jonathan Neuhauser requested to merge joneuhauser/extensions:reverse-multiple-subpaths into master Feb 10, 2022
  • Overview 5
  • Commits 1
  • Pipelines 2
  • Changes 2

What does the merge request do?

Correct the path command reversal of the absolute and relative move commands.

Closes #445 (closed)

Implementation notes

The unit tests only reversed paths with just one subpath (i.e. the Move command only occured as first command of a tested path), so the error didn't occur.

The example from #445 (closed) as well as a more complex example (see following picture) have been added as unit tests.

grafik

Summary for release notes

The "Reverse Path" method now returns correct results for paths with multiple subpaths.

Checklist

  • Add unit tests (if applicable)
  • Changes to inkex/ are well documented
  • Clean merge request history
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: reverse-multiple-subpaths