I would like to use ToCss for the Selector. Selector is basically a wrapper of the cssparser::SelectorList which implements this trait too.
SelectorList is containing exactly the content of Selector::selectors but with redeclaring it and requiring the explicit dependency on smallvec. Changing this removes smallvec as a direct dependency. This should even have the same memory footprint as Rust optimizes the wrapper struct away.
Using pub for the use ToCSS is probably debatable but I think its a good thing to easily allow to use to_css and to_css_string without specifying the cssparser dependency explicitly and rather use scraper::selector::ToCss instead.
cfvescovo
commented
8 months ago
I would like to use ToCss for the
Selector.Selectoris basically a wrapper of thecssparser::SelectorListwhich implements this trait too.SelectorListis containing exactly the content ofSelector::selectorsbut with redeclaring it and requiring the explicit dependency onsmallvec. Changing this removessmallvecas a direct dependency. This should even have the same memory footprint as Rust optimizes the wrapper struct away.Using
pubfor theuse ToCSSis probably debatable but I think its a good thing to easily allow to useto_cssandto_css_stringwithout specifying the cssparser dependency explicitly and rather usescraper::selector::ToCssinstead.