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.
I would like to use ToCss for the
Selector
.Selector
is basically a wrapper of thecssparser::SelectorList
which implements this trait too.SelectorList
is containing exactly the content ofSelector::selectors
but with redeclaring it and requiring the explicit dependency onsmallvec
. Changing this removessmallvec
as a direct dependency. This should even have the same memory footprint as Rust optimizes the wrapper struct away.Using
pub
for theuse ToCSS
is probably debatable but I think its a good thing to easily allow to useto_css
andto_css_string
without specifying the cssparser dependency explicitly and rather usescraper::selector::ToCss
instead.