#4965 closed enhancement (wontfix)

Opened November 18, 2009 06:49PM UTC

Closed July 30, 2010 10:22AM UTC

Last modified October 03, 2012 05:06PM UTC

Pull up selector-escaping from tabs into ui.core

Reported by:	jzaefferer	Owned by:
Priority:	critical	Milestone:	1.8
Component:	ui.core	Version:	1.7.2
Keywords:		Cc:
Blocked by:		Blocking:

Description

And while at it, do it properly. Currently tabs escapes only colons, while other characters like dots or square brackets should be escaped as well.

Marking this as critical as it will fix one or more tabs tickets and should be considered to inclusion in jQuery core. The earlier we land it, the higher the odds that it eventually lands in jQuery core.

Attachments (0)

Change History (4)

Changed January 20, 2010 04:50PM UTC by jzaefferer comment:1

description:	And while at it, do it properly. Currently tabs escapes only colons, while other characters like dots or square brackets should be escaped as well. \ \ Marking this as critical as it will fix one or more tabs tickets and should be considered to inclusion in jQuery core. The earlier we land it, the higher the odds that it eventually lands in jQuery core. → And while at it, do it properly. Currently tabs escapes only colons, while other characters like dots or square brackets should be escaped as well. \ \ Marking this as critical as it will fix one or more tabs tickets and should be considered to inclusion in jQuery core. The earlier we land it, the higher the odds that it eventually lands in jQuery core. \ \ See also http://dev.jqueryui.com/ticket/2715

description:

And while at it, do it properly. Currently tabs escapes only colons, while other characters like dots or square brackets should be escaped as well. \ \ Marking this as critical as it will fix one or more tabs tickets and should be considered to inclusion in jQuery core. The earlier we land it, the higher the odds that it eventually lands in jQuery core. → And while at it, do it properly. Currently tabs escapes only colons, while other characters like dots or square brackets should be escaped as well. \ \ Marking this as critical as it will fix one or more tabs tickets and should be considered to inclusion in jQuery core. The earlier we land it, the higher the odds that it eventually lands in jQuery core. \ \ See also http://dev.jqueryui.com/ticket/2715

Changed January 28, 2010 02:02AM UTC by scottgonzalez comment:2

milestone:	1.8 → 1.next

Changed July 30, 2010 10:22AM UTC by jzaefferer comment:3

resolution:	→ wontfix
status:	new → closed

Introducing a new method for a rather simple regex-replace isn't really worth it. If we need it anywhere else, we can still introduce that.

Changed October 03, 2012 05:06PM UTC by scottgonzalez comment:4

milestone:	1.next → 1.8

Search and Top Navigation