Bug Tracker

Opened 12 years ago

Closed 9 years ago

Last modified 8 years ago

#2662 closed bug (worksforme)

attr "action" of form and Selectors' attribute filter

Reported by: vytautas Owned by: flesler
Priority: low Milestone:
Component: selector Version: 1.2.3
Keywords: Cc:
Blocked by: Blocking:

Description

html:

<form action="account/edit">....</form>

javascript:

$('form[action="account/edit"]').size();
$('form[action="account/edit"]').attr('action');
$('form[action="http://example.org/account/edit"]').size();
$('form[action="http://example.org/account/edit"]').attr('action'); 

results:

firefox-1.5.0.1, firefox-2.0.0.13, firefox-3b5: 0, "undefined", 1, "account/edit"
mozilla-1.7.7, opera-9.50b, ie-6, ie-7, safari-3.1.525.13: 1, "account/edit", 0, "undefined"
opera-8.0, opera-9.00, opera-9.26: 0, "undefined", 1, "http://example.org/account/edit"

conclusions:

- the firefoxes differ
- jquery should hide these browser incompatibilities and always result in: 1, "account/edit", 0, "undefined"

Attachments (2)

action.2.html (1.1 KB) - added by vytautas 12 years ago.
action.html (1.1 KB) - added by vytautas 12 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 12 years ago by flesler

Resolution: fixed
Status: newclosed

This needs to be retested, but I think this is fixed now at [5574].

Please reopen if necessary.

comment:2 Changed 12 years ago by vytautas

Resolution: fixed
Status: closedreopened

Reopening, because nothing changed.

Here are test results using jquery-1.2.6:

firefox-1.5.0.1, firefox-2.0.0.14, firefox-3_rc1: 0, "undefined", 1, "account/edit" mozilla-1.7.7, opera-9.50b, ie-6, ie-7, safari-3.1.1_525.17.0: 1, "account/edit", 0, "undefined" opera-8.0, opera-9.00, opera-9.27: 0, "undefined", 1, "http://example.org/account/edit"

While all browsers should return: 1, "account/edit", 0, "undefined"

Changed 12 years ago by vytautas

Attachment: action.2.html added

Changed 12 years ago by vytautas

Attachment: action.html added

comment:3 Changed 12 years ago by flesler

Owner: set to flesler
Status: reopenednew

comment:4 Changed 11 years ago by flesler

Component: coreselector
Milestone: 1.2.41.3

#3194 fixes this, but adds a lot of overhead... This will require hard work with the benchmarker.

comment:5 Changed 11 years ago by dmethvin

See also #3113 and #3245.

comment:6 Changed 9 years ago by Rick Waldron

Keywords: needsreview added
Milestone: 1.3
Priority: majorlow
Status: newopen

Was this implemented?

comment:7 Changed 9 years ago by dmethvin

Resolution: worksforme
Status: openclosed

http://jsfiddle.net/dmethvin/6rCGW/ IE 8, FF 3.5, Chrome 7, Opera 10, Safari5 give identical results with the 1.4.4pre GIT0 version.

comment:8 Changed 9 years ago by dmethvin

Keywords: needsreview removed
Note: See TracTickets for help on using tickets.