Added Id Prefix/Suffix plugin #813
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
I was thinking about a random property. Also it should prefix only id + links. What is the goal of supporting suffix / prefix.
I am sorry but like that the feature does not solve any problem. I will try to give a concrete spec of what need to be implemented to avoid missed PR.
|
There are some use cases where allowing the user to specify the prefix are important, eg if some css styles are done via id's. Although In the name of simplicity I am happy to rework the plugin to just generate a random prefix for each id |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Summary
This plugin allows users to pass a prefix / suffix prop for id's. This enables you to have multiple and identical svg's without issue.
A related issue can be found here: #731
The plugin picks up on id's as well as references to id's. For example, when referencing a mask via
mask="url(#maskid)Test plan
I have written unit tests for this plugin. No other testing was done since the changes are only scoped to this new plugin
What's next
I am not aware of how this repo is set up so I'll need help finishing off the integration + writing docs.