Bug Tracker

Opened 10 years ago

Closed 10 years ago

#13801 closed feature (fixed)

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: #14341

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.

Change History (14)

comment:1 Changed 10 years ago by m_gol

+1

comment:2 Changed 10 years ago by timmywil

Component: unfiledcore
Keywords: 1.10-discuss added
Priority: undecidedlow
Status: newopen

+1, we always use ownerDocument anyway.

comment:3 Changed 10 years ago by Rick Waldron

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

comment:4 Changed 10 years ago by dmethvin

+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?

comment:5 Changed 10 years ago by gibson042

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

comment:6 Changed 10 years ago by timmywil

Milestone: None1.11/2.1

comment:7 Changed 10 years ago by timmywil

Keywords: 1.11-discuss added; 1.10-discuss removed

comment:8 Changed 10 years ago by gibson042

Keywords: 1.10-discuss added

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

comment:10 Changed 10 years ago by m_gol

Agreed.

comment:12 Changed 10 years ago by m_gol

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. :)

comment:13 Changed 10 years ago by m_gol

Resolution: fixed
Status: openclosed

comment:14 Changed 8 years ago by gibson042

Blocking: 14341 added
Note: See TracTickets for help on using tickets.