Missing bbox() method in Use class

meerk40t / svgelements

SVG Parsing for Elements, Paths, and other SVG Objects.

MIT License

127 stars 28 forks source link

Missing bbox() method in Use class #197

Closed ptomulik closed 1 year ago

ptomulik commented 1 year ago

Is it intentional? Use should be pretty similar to Group in this matter. Currently it just throws NotImplementedError.

tatarize commented 1 year ago

No. It's an oversight. I added Use fully in 1.8.0 (two versions back) and according to the class diagram the Use is very much like a Group but isn't one. Both are structural nodes so they got flagged as similar but I didn't check everything Use should have or port that over from group.

ptomulik commented 1 year ago

as #198 fixes this issue, I think I can close it.