Bug Tracker

Opened 11 years ago

Closed 8 years ago

#3245 closed bug (fixed)

Can not select a form using the action attribute

Reported by: mw44118 Owned by: flesler
Priority: minor Milestone: 1.3
Component: selector Version: 1.2.6
Keywords: selector form action attribute Cc:
Blocked by: Blocking:

Description

I had a form like

<form action="xyz"> </form>

I tried to select it with

$("form[action='xyz']")

and I got back an empty set of nodes.

Attachments (2)

3245.html (1.8 KB) - added by nathanhammond 11 years ago.
Correlating Info
special.html (6.3 KB) - added by nathanhammond 11 years ago.
Test case for href, action, and src attributes.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 11 years ago by ThrushAAX

however $("form[action*='xyz']") does work...

comment:2 Changed 11 years ago by flesler

Component: coreselector
Owner: set to flesler
Status: newassigned

This is related to another ticket... but I don't recall the number right now. We'll try to fix this for 1.3 (without hitting on perfomance).

comment:3 Changed 11 years ago by nathanhammon

Related to 2747 I think.

Changed 11 years ago by nathanhammond

Attachment: 3245.html added

Correlating Info

comment:4 Changed 11 years ago by nathanhammon

This is actually a separate bug from 2747. To make this one consistent everywhere we only have to add 'action' into the list of attributes that are special to force them to be accessed using getAttribute('action'). Accessing Element.action (DOM level 0) pulls back a fully qualified domain in IE8 and FF. Attaching a patch.

comment:5 Changed 11 years ago by nathanhammon

So I was wrong. Something a little funky is going on. I'm attaching a complete test comparison for href, src, and action instead.

Changed 11 years ago by nathanhammond

Attachment: special.html added

Test case for href, action, and src attributes.

comment:6 Changed 11 years ago by dmethvin

See also #3113 and #2662.

comment:7 Changed 8 years ago by danheberden

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.