Skip to main content

Bug Tracker

Side navigation

#13801 closed feature (fixed)

Opened April 20, 2013 06:01PM UTC

Closed May 28, 2013 01:15PM UTC

Deprecate context property

Reported by: gibson042 Owned by:
Priority: low Milestone: 1.10
Component: core Version: 1.9.1
Keywords: 1.10-discuss 1.11-discuss Cc:
Blocked by: Blocking:
Description

#9469 made a good (and ultimately successful) case for deprecating the selector property because of its needlessness and inherent unreliability. We should do the same with its sibling property context for exactly the same reasons.

Cf. https://github.com/jquery/jquery/pull/1249, closed because context is still publicly documented.

Attachments (0)
Change History (14)

Changed April 20, 2013 06:06PM UTC by m_gol comment:1

+1

Changed April 22, 2013 01:41PM UTC by timmywil comment:2

component: unfiledcore
keywords: → 1.10-discuss
priority: undecidedlow
status: newopen

+1, we always use ownerDocument anyway.

Changed April 22, 2013 01:46PM UTC by rwaldron comment:3

+1, as long as the iframe cases are served :)

Changed April 22, 2013 02:27PM UTC by dmethvin comment:4

+1, Good with the deprecation/removal but not the time frame. 2.0 has shipped, and 1.10 should parallel it. Do you mean 2.1/1.11 here?

Changed April 22, 2013 02:34PM UTC by gibson042 comment:5

2.1/1.11 is the most realistic. It'd be nice to get this sooner, but 2.0 has already left the barn.

Changed May 13, 2013 04:12PM UTC by timmywil comment:6

milestone: None1.11/2.1

Changed May 13, 2013 04:12PM UTC by timmywil comment:7

keywords: 1.10-discuss1.11-discuss

Changed May 13, 2013 04:14PM UTC by gibson042 comment:8

keywords: 1.11-discuss1.10-discuss 1.11-discuss

I think this was miscommunicated. Let's deprecate it now (1.10) and remove the code when we can.

Changed May 13, 2013 04:20PM UTC by gibson042 comment:9

Changed May 13, 2013 04:31PM UTC by m_gol comment:10

Agreed.

Changed May 21, 2013 07:38AM UTC by m_gol comment:11

Changed May 22, 2013 01:48PM UTC by m_gol comment:12

We have a race condition here. ;) The pull request is almost ready to merge but I need the URL to the blog post about the 1.10 release so we'll wait for the release with merging that. :)

Changed May 28, 2013 01:15PM UTC by m_gol comment:13

resolution: → fixed
status: openclosed

Changed December 07, 2014 04:30AM UTC by gibson042 comment:14

blocking: → 14341