Should ::part() be a String instead of a Selector?

[WilcoFiers]

I noticed this change added recently: https://github.com/mdevils/css-selector-parser/commit/6dbe1320cc4c06e038188b7a18497bac6ecc48a7

Looking over the spec ::part() takes a space separated list of identifiers, not a CSS selector. I think those probably could be parsed as a selector, but that doesn't seem to be the intent.

I was hoping I could correct it by adding String: ['part'] but this results in a Conflicting pseudo-class argument type: "Selector" vs "String". error. Is there a way to remove an pseudo element from a particular type that I'm not seeing, or is this extend method only additive? The syntax definitions don't seem to be exported so I can't modify them manually either.

[mdevils]

Hello @WilcoFiers , nice catch.

Feel free to open a PR to fix this.

Remove it from here: https://github.com/mdevils/css-selector-parser/blob/master/src/syntax-definitions.ts#L406

And change from Selector: ['part'] to String: ['part'] here: https://github.com/mdevils/css-selector-parser/blob/master/src/syntax-definitions.ts#L492