Standardize custom pseudo object #2073
Comments
|
I actually would like to keep them all. Regardless, I'd like to continue recommending the use of the .createPseudo() method for creating custom selectors, making the choice of one property somewhat moot – except for |
In the sense of not moving to deprecated, or of not standardizing?
But still exceedingly rare from (among other things) its lack of documentation, and with a fix that couldn't be simpler. At any rate, though, I propose deprecation not because I think we can remove them anytime soon, but to promote consistency.
That would depend on your measurement criteria... it's probably the most used by execution count from jQuery UI alone. And to the extent that any such special-case functionality can be used "much", I bet updating api.jquery.com would address the educational gap.
Then it appears NWMatcher has moved on, at least from an API perspective (which is what I'm addressing). However, even if it hadn't, maintaining surface area in jQuery core to support transitioning from an independent selector engine seems... overaccommodating.
Not at all; the result of |
I think we're on different wavelengths here. In terms of defining custom pseudos, you said jQuery UI mostly used
Totally true. Brain fart. However, the main point there was I'm all for documenting custom pseudo creation in jQuery (and I'm fine picking a property and sticking with it), but I still don't see the value behind deprecating the other properties. To me, deprecation voices the intention of removal whether we say it does or not. If we use the same property in all of our documentation, that seems like enough. |
|
Per meeting, decided on "pseudos". |
cc @timmywil @scottgonzalez
The Sizzle pseudos object is exposed on
jQuery.expras three distinct properties:jQuery.expr.pseudosjQuery.expr.filtersjQuery.expr[":"]Sizzle defines the first two, and jQuery defines the third.
None of them appear in the current jQuery documentation, though Sizzle documents the object as
pseudos(and separately documents the relatedsetFilters), and I'm sure we have published instructions on how to define a custom pseudo at least once, which would necessarily reference one of them. We also have internal uses in at least two libraries:jQuery.expr[":"]fromjQuery.expr.pseudosjQuery.expr.filtersjQuery.expr.filters.hiddenin the :visible pseudo$.expr[ ":" ]$.expr.filters.animated$.expr.filters.visibleand$.expr.filters.hiddenSo now that this issue affects three source repositories and jquery/api.jquery.com, here's the crux of it: can we standardize on one internally and deprecate the others? All things considered, my vote would be on
filters.The text was updated successfully, but these errors were encountered: