Bug Tracker

Ticket #13420 (closed bug: fixed)

Opened 22 months ago

Last modified 18 months ago

jQuery 1.9.1 fails to filter SVG parent nodes by class name when using .parent() and .closest()

Reported by: brad@… Owned by:
Priority: low Milestone: 1.10/2.0
Component: selector Version: git
Keywords: Cc:
Blocking: Blocked by:

Description

When using the parent and closest methods to walk up an SVG DOM tree in jQuery 1.9.1, filtering by selector does not work.

 http://jsfiddle.net/DruHG/

It looks *possible* Sizzle changed their XML element detection...

Change History

comment:1 Changed 22 months ago by gibson042

  • Priority changed from undecided to low
  • Resolution set to duplicate
  • Status changed from new to closed
  • Component changed from unfiled to selector

Duplicate of #12626.
SVG is not HTML; use [class~="name"]:  http://jsfiddle.net/DruHG/3/

comment:2 Changed 22 months ago by anonymous

Thank you, I do understand this.

It is strange that maintaining a predictable API is taking a back seat here.

I went through the duplicates, this issue is re-occurring and this XML feature seems to appear and disappear in various jQuery versions. It works one day and not the next with "wont fix" as the resolution. Why not just fix it?

My bad for the duped report though. Appreciate you guys.

comment:3 Changed 22 months ago by gibson042

  • Status changed from closed to reopened
  • Resolution duplicate deleted
  • Milestone changed from None to 1.9.2

I understand and appreciate that viewpoint, but SVG support is peripheral to our primary purpose with jQuery core and thus not guaranteed, though we do incorporate fixes on a case-by-case basis when their size/performance impact is sufficiently small.

The  jQuery SVG plugin attempts to provide better support, and you'll be happy to note that your particular application is in fact functional with the latest Sizzle update via jquery-git:  http://jsfiddle.net/DruHG/4/

comment:4 Changed 22 months ago by gibson042

  • Status changed from reopened to closed
  • Resolution set to fixed

comment:5 Changed 22 months ago by brad@…

This is excellent. I appreciate your time.

comment:6 Changed 18 months ago by dmethvin

  • Milestone changed from 1.9.2 to 1.10/2.0

Bulk update to milestone 1.10/2.0

Note: See TracTickets for help on using tickets.