Opened 8 years ago
Last modified 4 years ago
#16364 assigned task
Deserialized shapes don't keep their IDs
Reported by: | gaxweb | Owned by: | Eric Durocher |
---|---|---|---|
Priority: | undecided | Milestone: | 1.15 |
Component: | DojoX GFX | Version: | 1.8.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
If you deserialize an SVG which was converted with the svg2gfx.xsl stylesheet, the generated "name" property in the JSON is not being converted into a id attribute on the svg element it represents.
This makes it very hard to selectively add events on the graphic, for example. So i suggest to add that.
Attachments (1)
Change History (7)
Changed 8 years ago by
Attachment: | keep_IDs.patch added |
---|
comment:1 Changed 8 years ago by
The patch will check if an element inside the graphic already has used that ID and will then generate a new one instead, just like it does now.
comment:2 Changed 8 years ago by
Owner: | changed from Eugene Lazutkin to Patrick Ruzand |
---|---|
Status: | new → assigned |
comment:3 Changed 8 years ago by
Owner: | changed from Patrick Ruzand to Eric Durocher |
---|
comment:4 follow-up: 5 Changed 5 years ago by
Milestone: | tbd → 1.12 |
---|
Not sure why this patch was never reviewed. gaxweb, do you have a CLA on file ( see contributing guidelines at https://github.com/dojo/dojo/blob/master/CONTRIBUTING.md )?
We will consider this for 1.12.
comment:5 Changed 4 years ago by
Replying to dylan:
Not sure why this patch was never reviewed. gaxweb, do you have a CLA on file ( see contributing guidelines at https://github.com/dojo/dojo/blob/master/CONTRIBUTING.md )?
No, I do not. I shall talk to my boss.
comment:6 Changed 4 years ago by
Milestone: | 1.13 → 1.15 |
---|
Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.
A patch that does that.