Bug Tracker

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#8475 closed bug (invalid)

this :not() selector doesn't seem to work for me

Reported by: daniel.mendelsohn@… Owned by: daniel.mendelsohn@…
Priority: low Milestone: 1.next
Component: selector Version: 1.5.1
Keywords: Cc:
Blocked by: Blocking:

Description

$( "#iw-claim-form input[ type='text' ]:not( [ disabled='disabled' ] )" ).each();

in IE 8 this still iterates over the disabled inputs

works fine in FF 3.6 and chrome 9

thanks

markup is as follows

<div id="iw-claim-form">

<div id="error-list">

Fix your error stupid!

</div>

<form id="claim-form" action="" method="">

<div class="side-group top-group clearfix">

<div class="side-by-side">

<label for="claim-code">Claim Code:</label> <input type="text" name="claim-code" id="claim-code" value="some code junk" disabled="disabled" />

</div> <div class="side-by-side">

<label for="email">Email:</label> <input type="text" name="email" id="email" value="some code junk" disabled="disabled" />

</div>

</div>

<label for="title" >Title:</label> <select name="title" id="title" tabindex="1">

<option>Title</option>

<option>Mr.</option> <option>Mrs.</option> <option>Ms.</option>

</select>

<div class="side-group clearfix">

<div class="side-by-side">

<label for="first-name">First Name:</label> <input type="text" name="first-name" id="first-name" maxlength="50" size="35" value="First Name" tabindex="2" />

</div> <div class="side-by-side">

<label for="last-name">Last Name:</label> <input type="text" name="last-name" id="last-name" maxlength="50" size="35" value="Last Name" tabindex="3" />

</div>

</div>

Change History (8)

comment:1 Changed 8 years ago by anonymous

here is the example in jsFiddle http://jsfiddle.net/NjpzW/

comment:2 Changed 8 years ago by Rick Waldron

Owner: set to daniel.mendelsohn@…
Status: newpending

Thanks for taking the time to contribute to the jQuery project! Please provide a REDUCED jsFiddle test case to help us assess your ticket!

Additionally, test against the jQuery 0 GIT version to ensure the issue still exists.

comment:3 Changed 8 years ago by Rick Waldron

Component: unfiledselector

comment:4 Changed 8 years ago by daniel.mendelsohn@…

Status: pendingnew

here is the reduced jsFiddle version http://jsfiddle.net/f6aaU/

not quite sure how to test again 0 GIT

anyway, multiple different outcomes for this in IE8 and IE 7

comment:5 Changed 8 years ago by dmethvin

I believe there's an open ticket for this already that I can't find, Sizzle is looking at the property rather than the attribute. As a workaround try this selector:

"#foo input[type='text']:not( :disabled ), #foo select"

comment:6 Changed 8 years ago by Rick Waldron

Priority: undecidedlow
Status: newpending

comment:7 Changed 8 years ago by trac-o-bot

Resolution: invalid
Status: pendingclosed

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!

comment:8 Changed 8 years ago by daniel.mendelsohn@…

Yes this still does not work and i believe it would be cool if you guys would fix it since you say you are CSS3 selectors compliant but in this instant your not 100% compliant in IE

not being a jerk, i love jQuery and want to help it be even more awesome and perfect

should this be posted to a sizzle bug?

basically :not used in this example is not working in IE8 and lower http://jsfiddle.net/f6aaU/

Note: See TracTickets for help on using tickets.