Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit 05cf606

Browse files
mgolSplaktar
andauthored
fix(jqLite): apply suggestions from code review
Co-authored-by: Michael Prentice <splaktar@gmail.com>
1 parent 2df43c0 commit 05cf606

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/Angular.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -1959,10 +1959,10 @@ function bindJQuery() {
19591959
* @description
19601960
* Restores the pre-1.8 behavior of jqLite that turns XHTML-like strings like
19611961
* `<div /><span />` to `<div></div><span></span>` instead of `<div><span></span></div>`.
1962-
* The new behavior is a security fix so if you use this method, please try to adjust
1963-
* to the change & remove the call as soon as possible.
1962+
* The new behavior is a security fix. Thus, if you need to call this function, please try to adjust
1963+
* your code for this change and remove your use of this function as soon as possible.
19641964
1965-
* Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read
1965+
* Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read the
19661966
* [jQuery 3.5 upgrade guide](https://jquery.com/upgrade-guide/3.5/) for more details
19671967
* about the workarounds.
19681968
*/

src/jqLite.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@
9696
* ```js
9797
* angular.UNSAFE_restoreLegacyJqLiteXHTMLReplacement();
9898
* ```
99-
* Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read
99+
* Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read the
100100
* [jQuery 3.5 upgrade guide](https://jquery.com/upgrade-guide/3.5/) for more details
101101
* about the workarounds.
102102
*
@@ -196,7 +196,7 @@ wrapMap.th = wrapMap.td;
196196

197197
// Support: IE <10 only
198198
// IE 9 requires an option wrapper & it needs to have the whole table structure
199-
// set up up front; assigning `"<td></td>"` to `tr.innerHTML` doesn't work, etc.
199+
// set up in advance; assigning `"<td></td>"` to `tr.innerHTML` doesn't work, etc.
200200
var wrapMapIE9 = {
201201
option: [1, '<select multiple="multiple">', '</select>'],
202202
_default: [0, '', '']

0 commit comments

Comments
 (0)