Bug Tracker

Modify ↓

Ticket #3033 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

Useless check ?

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

Description

Line 269 of core.js: 

return this.pushStack( /[^+>] [^+>]/.test( selector ) || selector.indexOf("..") > -1 ?

It tests for the '..' selector. We dropped this option long ago. Though.. some people might be using compatibility plugins (I'm actually doing that in a project).

We could remove this part and add it to a future compatibility plugin.

Change History

comment:1 Changed 5 years ago by flesler

  • Status changed from new to assigned

comment:2 Changed 5 years ago by john

Yep, we can definitely drop this for 1.3.

comment:3 Changed 5 years ago by flesler

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

Applied at [5775].

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.