Side navigation
#7567 closed bug (invalid)
Opened November 19, 2010 10:14AM UTC
Closed December 04, 2010 08:05AM UTC
jQuery passes too much elements to Sizzle.filter in Internet Explorer
Reported by: | thereloaded | Owned by: | thereloaded |
---|---|---|---|
Priority: | undecided | Milestone: | 1.5 |
Component: | unfiled | Version: | 1.4.4 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
In Internet Explorer more elements are passed and matched in Sizzle.filter than actually exist on the page.
I made a testcase. Its not really beautiful, but maybe enough to illustrate: http://jsfiddle.net/VySgC/4/
Just click on the firebug-icon in the right-bottom corner to see the results. (sometimes the console is empty, dunno why, try to use Ctrl+F5).
Attachments (0)
Change History (8)
Changed November 19, 2010 10:18AM UTC by comment:1
owner: | → thereloaded |
---|---|
status: | new → pending |
Changed November 19, 2010 01:50PM UTC by comment:2
uhm. what doesnt work? works perfect in any browser (ie7, ie8, ff, safari, chrome, opera) and any computer i've access.
Changed November 19, 2010 01:57PM UTC by comment:3
status: | pending → new |
---|
ok, i guess my description is maybe not enough.
In firefox only one element is passed to Sizzle.filter and in internet explorer 8 and opera there are 4 elements. for firefox the one element doesnt match. for ie8 there are 4 matches for CLASS, 1 for PSEUDO and 3 for TAG. In opera there are 3 matches for CLASS and 3 for TAG.
also is it allowed to use firebug-lite for my testcase. or do i have to echo the results to the resultwindow?
Changed November 19, 2010 02:57PM UTC by comment:4
_comment0: | There are several issues with your test case. \ \ 1) It seems to use a modified version of jQuery 1.4.4. You failed to even mentioned that. Nor did you explain what the modifications do. \ \ 2) The server where this modified version resides seems to return a 404 for this file for every other request which makes debugging very difficult. \ \ 3) The javascript code is wrapped twice \ \ {{{ \ window.onload=function(){ \ jQuery(document).ready(function(){ \ .... \ }) \ } \ }}} \ \ 4) The inner-workings of jQuery/Sizzle are non trivial and the results you see might have to do with different browsers quirks which jQuery handles for you. \ \ 5) It isn't really clear what you expect the output to be. \ \ So unless you can show specifically that/what jQuery/Sizzle does wrong in your opinion by providing a test where the result is wrong I guess it will be unlikely that someone is going to take the time to reverseengineer what your complaint is about. \ \ If you mean to report a "performance" issue (Opera/IE do more work then necessary when selecting stuff) you should even more go into detail where you think jQuery/Sizzle goes wrong → 1290178769121364 |
---|---|
_comment1: | There are several issues with your test case. \ \ 1) It seems to use a modified version of jQuery 1.4.4. You failed to even mentioned that. Nor did you explain what the modifications do. \ \ 2) The server where this modified version resides seems to return a 404 for this file for every other request which makes debugging very difficult. \ \ 3) The javascript code is wrapped twice \ \ {{{ \ window.onload=function(){ \ jQuery(document).ready(function(){ \ .... \ }) \ } \ }}} \ \ 4) The inner-workings of jQuery/Sizzle are non trivial and the results you see might have to do with different browsers quirks which jQuery handles for you. By e.g. having to call certain internal functions more often then you expect it to. \ \ 5) It isn't really clear what you expect the output to be. \ \ So unless you can show specifically that/what jQuery/Sizzle does wrong in your opinion by providing a test where the result is wrong I guess it will be unlikely that someone is going to take the time to reverseengineer what your complaint is about. \ \ If you mean to report a "performance" issue (Opera/IE do more work then necessary when selecting stuff) you should even more go into detail where you think jQuery/Sizzle goes wrong → 1290185363298580 |
status: | new → pending |
There are several issues with your test case.
1) It seems to use a modified version of jQuery 1.4.4. You failed to even mentioned that. Nor did you explain what the modifications do.
2) The server where this modified version resides seems to return a 404 for this file for every other request which makes debugging very difficult.
3) The javascript code is wrapped twice
window.onload=function(){ jQuery(document).ready(function(){ .... }) }
4) The inner-workings of jQuery/Sizzle are non trivial and the results you see might have to do with different browsers quirks which jQuery handles for you. By e.g. having to call certain internal functions more often then you expect it to.
5) It isn't really clear what you expect the output to be.
So unless you can show specifically that/what jQuery/Sizzle does wrong in your opinion by providing a test where the result is wrong I guess it will be unlikely that someone is going to take the time to reverseengineer what your complaint is about.
If you mean to report a "performance" issue (e.g. Opera/IE do more work then necessary when selecting stuff) you should even more go into detail where you think jQuery/Sizzle goes wrong
Changed November 19, 2010 04:19PM UTC by comment:5
status: | pending → new |
---|
sorry for the inconvenience, i tried to get some answers on the forum, but the only advice i got was to file a bugreport. i will rework my testcase and write a better documentation. thanks a lot.
Changed November 19, 2010 04:50PM UTC by comment:6
_comment0: | Replying to [comment:5 thereloaded]: \ > sorry for the inconvenience, i tried to get some answers on the forum, but the only advice i got was to file a bugreport. i will rework my testcase and write a better documentation. thanks a lot. \ \ No need to be sorry. If you could provide a clearer test case that would be great then we will further investigate the issue. → 1290185481506750 |
---|
Replying to [comment:5 thereloaded]:
sorry for the inconvenience, i tried to get some answers on the forum, but the only advice i got was to file a bugreport. i will rework my testcase and write a better documentation. thanks a lot.
No need to be sorry. If you could provide a clearer test case that would be great then we will further investigate the issue.
Adding the post you are where referring to
Changed November 19, 2010 10:23PM UTC by comment:7
status: | new → pending |
---|
Set to pending close until/unless bug reporter provides a workable test case.
Changed December 04, 2010 08:05AM UTC by comment:8
resolution: | → invalid |
---|---|
status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
Your test case does not work. Please provide a working, reduced test case showing only the bare minimum code necessary to illustrate your problem. Read the How to Report Bugs for more detailed instructions.