bumbu/svg-pan-zoom

Destroy does not reset element to its origins

Open

#171 opened on Dec 9, 2015

View on GitHub
 (9 comments) (0 reactions) (0 assignees)JavaScript (1,585 stars) (383 forks)batch import
bughelp wanted

Description

I have the case where I initialize and destroy an SVG Element based on its visibility and experienced, that the wrapping g with its transform="matrix(...);" attribute is not reset after destroying it. When then re-initializing the object, it will show the SVG at a veeery small scale as a starting point. Because the translation is then applied on top of the already existing matrix.

I tried to just remove the transform attribute by hand but that didn't produce my wanted result.

Any suggestions?

Contributor guide

Tech stack
javascript
Domain
frontend
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
needs investigation
Prerequisites
basic knowledge of SVGJavaScript
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50
Research direction
Investigate the destroy method implementation in the library (likely in the main source file, e.g., src/svg pan zoom.js). Look at how the transform attribute is handled on the SVG element or its wrapper group during initialization and destruction. Check comments in the issue for any reproduction steps or suggested fixes. Consider testing with a simple HTML page that toggles the SVG element's visibility to reproduce the scaling issue.