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.
Fixes #285 by removing a duplicative reference to the responsive foundation grid package.
The purpose of the
tools-cgb
package in Responsive Foundation is to only print the exact CSS classes your plugin needs. This includes all icons and grid classes. The proper way to change this behavior is to change the variables above it:Changing any of these values to "true" will print all classes out to the CSS, including all grid classes.
That said, that will re-enable this duplicative behavior. My strong suspicion is that we don't actually need all those classes. I don't see any references to the grid classes in the plugin except for in the Sass files, where placeholder classes are being used. I verified that this line was introduced here: ebd1a7d
This was probably to support extending the
.col-
classes. These have since been refactored to use placeholder classes, as intended, and it should now be safe to remove this import.