bumbu/svg-pan-zoom

<use [...] /> is cut off

Open

#52 opened on Jul 24, 2014

View on GitHub
 (11 comments) (0 reactions) (0 assignees)JavaScript (383 forks)batch import
bughelp wanted

Repository metrics

Stars
 (1,585 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

When I use symbols and uses to decrease the size of an svg because I use the same element multiple times I get some issues with svg-pan-zoom:

<?xml version="1.0" encoding="utf-8"?>
<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform" xmlns="http://www.w3.org/2000/svg">
<defs>
  <symbol id="my_id">
    <circle r="10" />
  </symbol>
</defs>
<g>
  <use xlink:href="#my_id"/>
</g>
<g transform="scale(1.5)">
  <use xlink:href="#my_id" x="2000" y="1000" />
</g>

When I use this svg on a screen that has not a height of 100px and a width of 2000px the second element is not visible. In my actual project I have paths that are displayed until the screen ends, but when I pan the svg up the path is cut off.

Contributor guide

Research direction
Examine how svg pan zoom handles SVG elements with transform attributes and <use> references. Check if the library calculates visible bounds correctly for elements that are not directly under the root SVG but nested in <g> with transforms. Investigate whether the library's pan boundaries consider the full transformed bounding box of <use> elements.
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.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitSVG
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60