Bug Tracker

#2662 closed bug (worksforme)

Opened April 06, 2008 04:31PM UTC

Closed November 11, 2010 01:23AM UTC

Last modified March 15, 2012 09:09AM UTC

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 June 04, 2008 12:53PM UTC.
  • action.html (1.1 KB) - added by vytautas June 04, 2008 12:54PM UTC.
Change History (8)

Changed May 13, 2008 12:47AM UTC by flesler comment:1

resolution: → fixed
status: new → closed

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

Please reopen if necessary.

Changed June 04, 2008 12:21PM UTC by vytautas comment:2

resolution: fixed
status: closed → reopened

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 June 04, 2008 03:40PM UTC by flesler comment:3

owner: → flesler
status: reopened → new

Changed July 28, 2008 05:28PM UTC by flesler comment:4

component: core → selector
milestone: 1.2.4 → 1.3

#3194 fixes this, but adds a lot of overhead...

This will require hard work with the benchmarker.

Changed February 08, 2009 06:34PM UTC by dmethvin comment:5

See also #3113 and #3245.

Changed October 29, 2010 05:08PM UTC by rwaldron comment:6

keywords: → needsreview
milestone: 1.3
priority: major → low
status: new → open

Was this implemented?

Changed November 11, 2010 01:23AM UTC by dmethvin comment:7

resolution: → worksforme
status: open → closed

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.

Changed November 11, 2010 01:24AM UTC by dmethvin comment:8

keywords: needsreview