Side navigation
#12201 closed bug (wontfix)
Opened August 06, 2012 04:57AM UTC
Closed August 06, 2012 06:21PM UTC
Sizzle filter api incompatible when adding a custom pseudo selector
| Reported by: | softlion@softlion.com | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | None |
| Component: | selector | Version: | 1.8rc1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
This code works on 1.7.2 and fails on 1.8 RC1 with error
Uncaught TypeError: Cannot read property '3' of undefined.
$.extend($.expr[':'], {
someOp: function(a, i, m, s) {
var dist = parseInt(m[3]);
//...
return false;
}
});
Attachments (0)
Change History (2)
Changed August 06, 2012 05:03AM UTC by comment:1
Changed August 06, 2012 06:21PM UTC by comment:2
| component: | unfiled → selector |
|---|---|
| priority: | undecided → low |
| resolution: | → wontfix |
| status: | new → closed |
The new way to create custom pseudos with arguments is much clearer and faster.
The documentation can be found here: https://github.com/jquery/sizzle/wiki/Sizzle-Documentation#wiki-pseudo-selectors.
Your example adjusted to conform to the new API: http://jsfiddle.net/timmywil/VuWNg/1/
Custom pseudos that don't require arguments work the same way.
jsFiddle demonstrating the problem:
http://jsfiddle.net/VuWNg/