Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new plugin: reduce id names to the minimum #7

Closed
deepsweet opened this issue Oct 17, 2012 · 2 comments
Closed

new plugin: reduce id names to the minimum #7

deepsweet opened this issue Oct 17, 2012 · 2 comments
Assignees
Milestone

Comments

@deepsweet
Copy link
Member

No description provided.

@ghost ghost assigned deepsweet Oct 17, 2012
@kizu
Copy link

kizu commented Oct 18, 2012

I can see it be not that safe: if someone would try to use SVG Stacks one day, the obfuscated IDs could break things up, 'cause author would no longer know what IDs are there.

However, if there are some common ID patterns used by SVG-producing software, it would be not that unsafe to reduce them.

However, I can see how an external Stylesheet could use those IDs. However, there can be a lot of other problems with external CSS, so I don't think it'd be a problem there :)

@deepsweet
Copy link
Member Author

yes, I thought about this issue, and probably it will be the first plugin disabled by default.
even is there are a lot of autogenerated ID like "SVG001" or "G57" there is no any guarantee that they are not used outside.

maybe svgo should ask questions like "do you use SVG id attrs somewhere? [y/n]" :)

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

No branches or pull requests

2 participants