Add removeAttributesBySelector plugin #929
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Removes attributes of elements that match a css selector. - Add plugin - Change config file to use plugin - Update README - Add tests
|
#977 Addresses the same issue utilizing the existing plugin. However, The plugin proposed in this PR has an extra benefit of being able to remove a different attribute than the one matching the selector. |
|
SVGO v1.2.0 has been just released! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request adds the
removeAttributesBySelectorplugin. Allowing the removal of attributes in elements found by a css selector.For a project I'm currently working on, I need to remove the attribute
fillonly if it has the value#00ff00. TheremoveAttrsplugin only queries the element name and is not able to meet this need.I'm happy to add other plugins like
addAttributesBySelector, or refactorremoveAttrs, or create another plugin that combines all this functionality. For now, I've just left it as a separate plugin.