Skip to content

Adds useDefsForGlyphs option#1087

Open
stewarthaines wants to merge 1 commit into
paulrosen:devfrom
stewarthaines:compact-dom
Open

Adds useDefsForGlyphs option#1087
stewarthaines wants to merge 1 commit into
paulrosen:devfrom
stewarthaines:compact-dom

Conversation

@stewarthaines

Copy link
Copy Markdown
  • new renderer option useDefsForGlyphs
  • handle as defs/use in svg dom when option is set
  • adds Svg.prototype.use() method
  • adds docs for new renderAbc() option useDefsForGlyphs
  • a couple of float precision numbers toFixed() for compact SVG attributes

* new renderer option useDefsForGlyphs
* handle as defs/use in svg dom when option is set
* adds Svg.prototype.use() method
* adds docs for new renderAbc() option useDefsForGlyphs
* a couple of float precision numbers toFixed() for compact SVG attributes
@stewarthaines

Copy link
Copy Markdown
Author

Hi @paulrosen

Is there something more you'd like to see in this PR?

I understand you have other priorities. If there's something obviously missing I would be happy to address it.

Thanks!

@paulrosen

Copy link
Copy Markdown
Owner

Hi, I'm sorry for the extreme delay! I'm juggling a few irons in the fire like a one-armed wall paper hanger.

I am interested this.

I need to dive in and try to figure out what complications might happen with this and do some testing and I just haven't had a chance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants