Remove Fizzler Dependency #1073
inforithmics
started this conversation in
Ideas
Replies: 2 comments 7 replies
-
I had a look at it, but it seems that CombinatorSelector needs to be made public. in Excss |
Beta Was this translation helpful? Give feedback.
1 reply
-
The licensing issue is not a problem IMO (I have closed the issue again, because the open issue of using ExCSS as a library has long been fixed). The author of the library has no problem with the usage, and the license permits the usage as is - so I see no real reason to replace it. |
Beta Was this translation helpful? Give feedback.
6 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I think I could make a Pull request to remove the Fizzler dependency, because this is the only place where it is used.
var elemsToStyle = rootNode.QuerySelectorAll(rule.Selector.Text, elementFactory);
And here the css selector is again parsed with fizzler, the rule.Selector is already a parsed css selector.
So this would have following benefits:
Beta Was this translation helpful? Give feedback.
All reactions