{"action":"created","issue":{"url":"https://api.github.com/repos/jquery/jquery/issues/2723","labels_url":"https://api.github.com/repos/jquery/jquery/issues/2723/labels{/name}","comments_url":"https://api.github.com/repos/jquery/jquery/issues/2723/comments","events_url":"https://api.github.com/repos/jquery/jquery/issues/2723/events","html_url":"https://github.com/jquery/jquery/issues/2723","id":117150910,"number":2723,"title":"2.2 Release progress","user":{"login":"markelog","id":945528,"avatar_url":"https://avatars.githubusercontent.com/u/945528?v=3","gravatar_id":"","url":"https://api.github.com/users/markelog","html_url":"https://github.com/markelog","followers_url":"https://api.github.com/users/markelog/followers","following_url":"https://api.github.com/users/markelog/following{/other_user}","gists_url":"https://api.github.com/users/markelog/gists{/gist_id}","starred_url":"https://api.github.com/users/markelog/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markelog/subscriptions","organizations_url":"https://api.github.com/users/markelog/orgs","repos_url":"https://api.github.com/users/markelog/repos","events_url":"https://api.github.com/users/markelog/events{/privacy}","received_events_url":"https://api.github.com/users/markelog/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/jquery/jquery/labels/2.x-only","name":"2.x-only","color":"fef2c0"},{"url":"https://api.github.com/repos/jquery/jquery/labels/Blocker","name":"Blocker","color":"f7c6c7"},{"url":"https://api.github.com/repos/jquery/jquery/labels/Needs%20review","name":"Needs review","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/jquery/jquery/milestones/9","html_url":"https://github.com/jquery/jquery/milestones/1.12/2.2","labels_url":"https://api.github.com/repos/jquery/jquery/milestones/9/labels","id":1378093,"number":9,"title":"1.12/2.2","description":"","creator":{"login":"mzgol","id":1758366,"avatar_url":"https://avatars.githubusercontent.com/u/1758366?v=3","gravatar_id":"","url":"https://api.github.com/users/mzgol","html_url":"https://github.com/mzgol","followers_url":"https://api.github.com/users/mzgol/followers","following_url":"https://api.github.com/users/mzgol/following{/other_user}","gists_url":"https://api.github.com/users/mzgol/gists{/gist_id}","starred_url":"https://api.github.com/users/mzgol/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mzgol/subscriptions","organizations_url":"https://api.github.com/users/mzgol/orgs","repos_url":"https://api.github.com/users/mzgol/repos","events_url":"https://api.github.com/users/mzgol/events{/privacy}","received_events_url":"https://api.github.com/users/mzgol/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":2,"state":"open","created_at":"2015-10-26T20:44:39Z","updated_at":"2015-12-02T02:21:03Z","due_on":null,"closed_at":null},"comments":17,"created_at":"2015-11-16T15:37:35Z","updated_at":"2015-12-07T18:32:33Z","closed_at":null,"body":"Instead of porting bug fixes to `2.2-stable` branch i decided to go with opposite way - reverting back-incompat commits.\r\n\r\nI tried more conventional approach at first (and another three too), but it proved (to me) to be ineffective. Since there was a lot more conflicts then with chosen approach which were prompted by the use of the new QUnit API and style fixes.\r\n\r\nBelow, you find three lists, one of the reverted commits and one of the questionable commits that are in `2.2-stable` and are back-compat, but might be inappropriate for this release. And also list of reverted commits which requires special attention.\r\n\r\nI would expect from reviewers to check those lists and current state of the branch, verifying that API doesn't include new features, don't break back-compat and done overall correctly.\r\n\r\nProcessed commits started from Oct 30, 2014.\r\n\r\n### Questionable commits\r\n\r\nDrop support for Opera 12.x, Firefox<29, Safari<6.0 and old Blackberry\r\nhttps://github.com/jquery/jquery/commit/740e190223d19a114d5373758127285d14d6b71e\r\n\r\nCallbacks module change, considered as a bug\r\nhttps://github.com/jquery/jquery/commit/89ce0af2cf7f001647e74fc1de92ce94a51fd5c2\r\n\r\nHides `jQuery.swap`\r\nhttps://github.com/jquery/jquery/commit/bb4d888f4f7886371347b59eae5d4e6135acb7ed\r\n\r\nAdds `jQuery.htmlPrefilter` - allowed by the semver for the minor version,\r\nbut might be not contracdict 2.2 release, also not documented.\r\nhttps://github.com/jquery/jquery/commit/225bde37c997f5ddd9fe00fdfb8e9a43545cfbbc\r\n\r\nChanges signature of the `jQuery#init`, needed for the Migrate (/cc @dmethvin)\r\nhttps://github.com/jquery/jquery/commit/7a6931de689865f559bed26e797e8cbb5674bbce\r\n\r\nAdds `jQuery.uniqueSort`, deprecates `jQuery.unique` - allowed by the semver for the minor version, but might be not contracdict 2.2 release, also not documented.\r\nhttps://github.com/jquery/jquery/commit/e1090c3d2b2a988a5b41f1f1ed9f8d6dcae02200\r\n\r\nHides `jQuery.domManip`\r\nhttps://github.com/jquery/jquery/commit/62d5579578109f1468a37e44f76af06f283a46ab\r\n\r\nHides `jQuery.buildFragment`\r\nhttps://github.com/jquery/jquery/commit/a74320fca8bb0a2190f6e1fdb71a73733b6986e4\r\n\r\nChanges signature (remove additional param) of `jQuery.event.simulate` couldn't find docs for it, if documented is not acceptable change for the minor\r\nhttps://github.com/jquery/jquery/commit/3655c4e1908cc3ee49487e7e26e8cfca9fe8146d\r\n\r\nHides `jQuery.acceptData`\r\nhttps://github.com/jquery/jquery/commit/224271982eb9cd351d7db1b38c740b4e927e6f97\r\n\r\nHides `jQuery.dir` and `jQuery.sibling`\r\nhttps://github.com/jquery/jquery/commit/f9ef427d355a8a2b531aed5d112dfa4f9692105c\r\n\r\nEven though subject of the commit reads \"Adding unit tests for jQuery.Animation\"\r\nThere was changes in logic and interface of the `effects` module (/cc @gnarf)\r\nhttps://github.com/jquery/jquery/commit/b3b2d6c3dd51fbdc69e1942e9af75cc99a1834c2\r\n\r\n### Reverted commits\r\n\r\nReason: API change (behaviour)\r\nName: Ajax: Always use script injection in globalEval\r\nCommit: https://github.com/jquery/jquery/commit/bbdfbb4ee859fe9319b348d88120ddc2c9efbd63\r\nIssue: trac-14757\r\nPR: https://github.com/jquery/jquery/pull/1449\r\n\r\nReason: API change (behaviour)\r\nName: Effects: Reintroduce use of requestAnimationFrame\r\nCommit: https://github.com/jquery/jquery/commit/72119e0023dcc0d9807caf6d988598b74abdc937\r\nIssue: trac-15147\r\nPR: https://github.com/jquery/jquery/pull/1578\r\n\r\nReason: Follow-up commit for\r\nName: Effects: Improve raf logic\r\nCommit: https://github.com/jquery/jquery/commit/708764f47b0c8de152bbb444d0f608db558b76ed\r\nPR: https://github.com/jquery/jquery/pull/1578\r\n\r\nReason: API change (removed methods)\r\nName: Deprecated: Drop size and andSelf methods\r\nCommit: https://github.com/jquery/jquery/commit/f110360f65a268e959ae892ca36e85da3d91e606\r\nIssue: https://github.com/jquery/jquery/issues/1749\r\nPR: https://github.com/jquery/jquery/pull/1822\r\n\r\nReason: Adherence to the latest version of DOM specification\r\nName: Manipulation: support data-URI scripts insertion\r\nCommit: https://github.com/jquery/jquery/commit/15f4dec7894f1e00adbfb9bce4f870441a527bd6\r\nIssue: https://github.com/jquery/jquery/issues/1887\r\nPR: https://github.com/jquery/jquery/pull/1888\r\n\r\nReason: API change (behaviour)\r\nName: Core: Throw an error on $(\"#\") rather than returning 0-length collection\r\nCommit: https://github.com/jquery/jquery/commit/80022c81ce4a07a232afd3c580b0977555a2daec\r\nIssue: https://github.com/jquery/jquery/pull/1682\r\n\r\nReason: Follow-up commit for \"Adherence to the latest version of DOM specification\"\r\nName: Manipulation: improve test for data-URI\r\nCommit: https://github.com/jquery/jquery/commit/dd596ccf729b2f39d44b73bc54b53bd41c880146\r\n\r\nReason: API change (behaviour)\r\nName: Manipulation: make wrapAll funarg execute only once\r\nCommit: https://github.com/jquery/jquery/commit/359b03cac74d7336676a6992f14b7ccab9b28659\r\nIssue: https://github.com/jquery/jquery/issues/1843\r\nPR: https://github.com/jquery/jquery/pull/1912\r\n\r\nReason: API change (behaviour)\r\nName: Callbacks: Don't abort execution on .lock()\r\nCommit: https://github.com/jquery/jquery/commit/32bf9178cace8c52a31c468a28adde9a4fdc4671\r\nIssue: https://github.com/jquery/jquery/issues/1990\r\nPR: https://github.com/jquery/jquery/pull/1991\r\n\r\nReason: API change (proprties removal)\r\nName: Core: Remove deprecated context and selector properties\r\nCommit: https://github.com/jquery/jquery/commit/0ea8c32863af31fb5cfc184e8d513bbae35583e8\r\nIssue: https://github.com/jquery/jquery/issues/1908\r\nPR: https://github.com/jquery/jquery/pull/2000\r\n\r\nReason: API change (return value)\r\nName: Core: Return empty array instead of null for parseHTML(\"\")\r\nCommit: https://github.com/jquery/jquery/commit/4116914dcac32868c2c822366785e3dd2ccc69d4\r\nIssue: https://github.com/jquery/jquery/issues/1997\r\nPR: https://github.com/jquery/jquery/pull/1998\r\n\r\nReason: API change (return value)\r\nName: Follow-up commit for \"Manipulation: support data-URI scripts insertion\"\r\nCommit: https://github.com/jquery/jquery/commit/4fae91141c04dffb5094b5961eb5d7b2a1f3f423\r\nIssue: https://github.com/jquery/jquery/issues/1993\r\n\r\nReason: API change (methods removal)\r\nName: Ajax: remove deprecated extensions from ajax promise\r\nCommit: https://github.com/jquery/jquery/commit/9d1b989f20b550af3590691723b0620f6914626e\r\nIssue: https://github.com/jquery/jquery/issues/2084\r\nPR: https://github.com/jquery/jquery/pull/2092\r\n\r\nReason: Promise interoperability\r\nName: Deferred: Backwards-compatible standards interoperability\r\nCommit: https://github.com/jquery/jquery/commit/555a50d340706e3e1e0de09231050493d0ad841e\r\nIssue: https://github.com/jquery/jquery/issues/1722\r\nPR: https://github.com/jquery/jquery/pull/1996\r\n\r\nReason: API change (feature)\r\nName: Wrap: Support .unwrap( selector) for selective unwrapping\r\nCommit: https://github.com/jquery/jquery/commit/7b09235ceed57bbcc26fc2c76147eb4e95ebdb92\r\nIssue: https://github.com/jquery/jquery/issues/1744\r\nPR: https://github.com/jquery/jquery/pull/2003\r\nNote: Could be included in the minor release, need documentation\r\n\r\nReason: API change (behaviour)\r\nName: Event: remove guard for falsy handler argument of jQuery#on method\r\nCommit: https://github.com/jquery/jquery/commit/fac67a984268ef8f7de952666fda6d8d32754f5f\r\nIssue: https://github.com/jquery/jquery/issues/2248\r\nPR: https://github.com/jquery/jquery/pull/2249\r\n\r\nReason: API change (behaviour)\r\nName: Data: do not include digits when camelCasing\r\nCommit: https://github.com/jquery/jquery/commit/2862a07af640be42711f72ad4abe532349985824\r\nIssue: https://github.com/jquery/jquery/issues/1751\r\n\r\nReason: API change (behaviour)\r\nName: Data: always camelCase keys in .data()\r\nCommit: https://github.com/jquery/jquery/commit/0e790985a76fd813a6e56696c87abeed5a6cf63b\r\nIssue: https://github.com/jquery/jquery/issues/2257\r\n\r\nReason: API change (behaviour)\r\nName: Data: camelCasing should not ignore case\r\nCommit: https://github.com/jquery/jquery/commit/172cad80ac635d8900aa6a3504c89f38b320488e\r\nIssue: https://github.com/jquery/jquery/issues/2070\r\n\r\nReason: API change (behaviour)\r\nName: Offset: Fix .offset() to correctly work with ShadowDOM\r\nCommit: https://github.com/jquery/jquery/commit/1617479fcf7cbdaf33dc9334ed10a0f30bf14687\r\nIssue: https://github.com/jquery/jquery/issues/1784\r\nPR: https://github.com/jquery/jquery/pull/2043\r\n\r\nReason: Follow-up commit for \"Offset: Fix .offset() to correctly work with ShadowDOM\"\r\nName: Offset: allow offset setter to throw for disconnected elements\r\nCommit: https://github.com/jquery/jquery/commit/0d11c1182f2012cd6eb06ce1e3fa5a495af9bee3\r\nIssue: https://github.com/jquery/jquery/issues/2114\r\n\r\nReason: Follow-up commit for \"Offset: Fix .offset() to correctly work with ShadowDOM\"\r\nName: Offset: remove ownerDocument check in offset getter\r\nCommit: https://github.com/jquery/jquery/commit/6176567361e9d05d4f5eb7e98735a678f2cd7ea9\r\nIssue: https://github.com/jquery/jquery/issues/2115\r\n\r\nReason: API change (behaviour)\r\nName: CSS: Ignore the CSS cascade in show()/hide()/etc.\r\nCommit: https://github.com/jquery/jquery/commit/86419b10bfa5e3b71a7d416288ab806d47a31d1f\r\nIssue: https://github.com/jquery/jquery/issues/1767\r\nIssue: https://github.com/jquery/jquery/issues/2071\r\nPR: https://github.com/jquery/jquery/pull/2180\r\n\r\nReason: Follow-up commit for \"CSS: Ignore the CSS cascade in show()/hide()/etc.\"\r\nName: CSS: Make show/hide/toggle methods a module\r\nCommit: https://github.com/jquery/jquery/commit/67d7a2eefee768b59eb3d51cb1fb2c671873e58a\r\nIssue: https://github.com/jquery/jquery/issues/2193\r\nPR: https://github.com/jquery/jquery/pull/2648\r\n\r\nReason: Follow-up commit for \"CSS: Ignore the CSS cascade in show()/hide()/etc.\"\r\nName: CSS: fix AMD mode for the new showHide module\r\nCommit: https://github.com/jquery/jquery/commit/32cfc38a9c76d1f9163759c811cb9c82eb47d565\r\n\r\nReason: Follow-up commit for \"CSS: Ignore the CSS cascade in show()/hide()/etc.\"  --\r\nName: Effects: fix loading showHide in AMD mode\r\nCommit: https://github.com/jquery/jquery/commit/ab06be561ec74cccaa2d581830210f82326f05c3\r\n\r\nReason: API change (event removal)\r\nName: Core: remove custom ready event\r\nCommit: https://github.com/jquery/jquery/commit/c252c5fac2a25869524c2eba2e93d4230e6af25c\r\nIssue: https://github.com/jquery/jquery/issues/2264\r\nPR: https://github.com/jquery/jquery/pull/2265\r\n\r\nReason: API change (events removal)\r\nName: Event: remove deprecated event aliases\r\nCommit: https://github.com/jquery/jquery/commit/0705be475092aede1eddae01319ec931fb9c65fc\r\nIssue: https://github.com/jquery/jquery/issues/2286\r\nPR: https://github.com/jquery/jquery/pull/2287\r\n\r\nReason: Follow-up commit for \"Event: remove deprecated event aliases\"\r\nName: Ajax: Remove remnants of the load event alias handling\r\nCommit: https://github.com/jquery/jquery/commit/38a669735d08bcbd28cfb0d77eee82c67aa89eeb\r\nPR: https://github.com/jquery/jquery/pull/2362\r\n\r\nReason: Revert of the API change\r\nName: Revert \"Offset: allow offset setter to throw for disconnected elements\"\r\nCommit: https://github.com/jquery/jquery/commit/578dcee96a8d4d759b3a7e623177fa36a5133ba7\r\n\r\nReason: API change (new method)\r\nName: Deferred: add .catch handler\r\nCommit: https://github.com/jquery/jquery/commit/84ccf2606c6b97d5875774bf774f9f2aae950ae7\r\nIssue: https://github.com/jquery/jquery/issues/2102\r\nNote: Could be included in the minor release, need documentation\r\n\r\nReason: API change (behaviour)\r\nName: CSS: Make .css(\"width\") & .css(\"height\") return fractional values\r\nCommit: https://github.com/jquery/jquery/commit/b60b26e18477d21fa5ec9c6572e114fc5d441730\r\nIssue: https://github.com/jquery/jquery/issues/1724\r\nPR: https://github.com/jquery/jquery/pull/2439\r\n\r\nReason: API change (signature)\r\nName: Remove additional parameters of easings\r\nCommit: https://github.com/jquery/jquery/commit/b7a7dea95f84d6d8e5a8186d4fb09a762baf79bb\r\nIssue: https://github.com/jquery/jquery/issues/2367\r\nPR: https://github.com/jquery/jquery/pull/2376\r\n\r\nReason: API change (behaviour)\r\nName: Ajax: Mitigate possible XSS vulnerability\r\nCommit: https://github.com/jquery/jquery/commit/b078a62013782c7424a4a61a240c23c4c0b42614\r\nIssue: https://github.com/jquery/jquery/issues/2432\r\nPR: https://github.com/jquery/jquery/pull/2588\r\n\r\nReason: Follow-up commit for \"Ajax: Mitigate possible XSS vulnerability\"\r\nName: Ajax: don't expect cross-origin tests run in envs which not support it\r\nCommit: https://github.com/jquery/jquery/commit/39cdb8c9aa0fde68f733553ba050a2ba9d86474c\r\n\r\nReason: API change (behaviour)\r\nName: Attributes: do not set properties to false when removing booleans\r\nCommit: https://github.com/jquery/jquery/commit/47ccf3daadc4b312f850502300129952e70f9d9d\r\nIssue: https://github.com/jquery/jquery/issues/1759\r\n\r\nReason: Follow-up commit for \"Attributes: do not set properties to false when removing booleans\"\r\nName: Attributes: remove flakey test for selected attribute\r\nCommit: https://github.com/jquery/jquery/commit/87bd130289c6ed9bfc355c1f8587ae6ce00a4776\r\n\r\nReason: API change (behaviour)\r\nName: Ajax: Only form-encode requests with a bod\r\nCommit: https://github.com/jquery/jquery/commit/70605c8e5655da996ebd395e3c43423daaa08d9c\r\nIssue: https://github.com/jquery/jquery/issues/2658\r\nPR: https://github.com/jquery/jquery/pull/2671\r\n\r\nReason: API change (behaviour)\r\nName: Attributes: return empty array for select-multiple with no values\r\nCommit: https://github.com/jquery/jquery/commit/79fc806e8500372a2278795c068d039ee287535f\r\nIssue: https://github.com/jquery/jquery/issues/2562\r\nPR: https://github.com/jquery/jquery/pull/2689\r\n\r\nReason: API change (behaviour)\r\nName: Dimensions: outerWidth/Height include scrollbar\r\nCommit: https://github.com/jquery/jquery/commit/7d44d7f9e7cb73ff2b373f08cea13ea9958bb462\r\nIssue: https://github.com/jquery/jquery/issues/1729\r\nPR: https://github.com/jquery/jquery/pull/2694\r\n\r\nReason: API change (return value)\r\nName: Dimensions: Empty sets should return undefined\r\nCommit: https://github.com/jquery/jquery/commit/97d79ecf6b6c5cc3c1485eb3c46e12986a978f57\r\n\r\nReason: API change (signature change)\r\nName: Manipulation: execute scripts from iframe in the iframe's context\r\nCommit: https://github.com/jquery/jquery/commit/22449eb968622c2e14d6c8d8de2cf1e1ba4adccd\r\nIssue: https://github.com/jquery/jquery/issues/1757\r\nPR: https://github.com/jquery/jquery/pull/2696\r\nNote: depends on \"Ajax: Always use script injection in globalEval\"\r\n\r\n### Complicated reverts\r\n\r\nCompatibility with standard Promises (/cc @gibson042)\r\nhttps://github.com/jquery/jquery/commit/2d9d1f006ac98b3dd1f23de1b721f54eb77346db\r\n\r\n`show()`/`hide()` changes (/cc @gibson042)\r\nhttps://github.com/jquery/jquery/commit/0169459d5d896a87e4cc0026bfd9b823d5525220\r\nhttps://github.com/jquery/jquery/commit/87db099ed54d0dac624c4ae6b476328e88aef2d7\r\nhttps://github.com/jquery/jquery/commit/88df7676d735b4cf16dd56ac55fbdef3319f66e7\r\nhttps://github.com/jquery/jquery/commit/43996d166ee72bc5180edeca2b3312d5c40709cd\r\n\r\n--\r\n\r\nYou might notice that i included new `jQuery#htmlPrefilter` but didn't `jQuery.Deferred#catch`.\r\nWhy? Because i think it might be a good opportunity to test new methods in the wild (would require a documentation updates though), but not the ones that are related to module rewrites (like `catch()`).\r\n\r\nIf this method would be found productive it could be applied to 1.x branch.\r\n\r\n/cc @jquery/core\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/jquery/jquery/issues/comments/162616532","html_url":"https://github.com/jquery/jquery/issues/2723#issuecomment-162616532","issue_url":"https://api.github.com/repos/jquery/jquery/issues/2723","id":162616532,"user":{"login":"gibson042","id":1199584,"avatar_url":"https://avatars.githubusercontent.com/u/1199584?v=3","gravatar_id":"","url":"https://api.github.com/users/gibson042","html_url":"https://github.com/gibson042","followers_url":"https://api.github.com/users/gibson042/followers","following_url":"https://api.github.com/users/gibson042/following{/other_user}","gists_url":"https://api.github.com/users/gibson042/gists{/gist_id}","starred_url":"https://api.github.com/users/gibson042/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gibson042/subscriptions","organizations_url":"https://api.github.com/users/gibson042/orgs","repos_url":"https://api.github.com/users/gibson042/repos","events_url":"https://api.github.com/users/gibson042/events{/privacy}","received_events_url":"https://api.github.com/users/gibson042/received_events","type":"User","site_admin":false},"created_at":"2015-12-07T18:32:32Z","updated_at":"2015-12-07T18:32:32Z","body":"> ## Commits in 2.1.4 not in 2.2-stable\r\n> \r\n> - e5190982c40d7ac8ab9bdb2e7e4334f0e123ef66 Ajax: Remove workaround for IE6/7\r\n> - 0cdec797de23555c95a70978f4d9e06f3b041330 Data: avoid non-alphanumeric chars in expando properties\r\n> - 4e7f34f6296111f7f91d621397dfb02c6bf4c41f Ajax: remove event dependency from the ajax module\r\n> - 7d15b4d1f181de8868c375c545a51e4dfc28a611 CSS: Clean up memory leak in reliableMarginRight\r\n> \r\n> Of these, I think only the last one is important. We probably want that memory leak gone in 2.1.4.\r\n\r\nI believe we want all of them except the first. 0cdec797de23555c95a70978f4d9e06f3b041330 avoids a performance regression, and 4e7f34f6296111f7f91d621397dfb02c6bf4c41f is good hygeine."}}