From 05cf60677b0cdac47ce6b860cbb7b41957a2cbba Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Tue, 26 May 2020 22:08:46 +0200 Subject: [PATCH] fix(jqLite): apply suggestions from code review Co-authored-by: Michael Prentice --- src/Angular.js | 6 +++--- src/jqLite.js | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/Angular.js b/src/Angular.js index 5df6836d9083..d7a68ccdb85f 100644 --- a/src/Angular.js +++ b/src/Angular.js @@ -1959,10 +1959,10 @@ function bindJQuery() { * @description * Restores the pre-1.8 behavior of jqLite that turns XHTML-like strings like * `
` to `
` instead of `
`. - * The new behavior is a security fix so if you use this method, please try to adjust - * to the change & remove the call as soon as possible. + * The new behavior is a security fix. Thus, if you need to call this function, please try to adjust + * your code for this change and remove your use of this function as soon as possible. - * Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read + * Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read the * [jQuery 3.5 upgrade guide](https://jquery.com/upgrade-guide/3.5/) for more details * about the workarounds. */ diff --git a/src/jqLite.js b/src/jqLite.js index 805fd23c9b5c..6efbadd10854 100644 --- a/src/jqLite.js +++ b/src/jqLite.js @@ -96,7 +96,7 @@ * ```js * angular.UNSAFE_restoreLegacyJqLiteXHTMLReplacement(); * ``` - * Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read + * Note that this only patches jqLite. If you use jQuery 3.5.0 or newer, please read the * [jQuery 3.5 upgrade guide](https://jquery.com/upgrade-guide/3.5/) for more details * about the workarounds. * @@ -196,7 +196,7 @@ wrapMap.th = wrapMap.td; // Support: IE <10 only // IE 9 requires an option wrapper & it needs to have the whole table structure -// set up up front; assigning `""` to `tr.innerHTML` doesn't work, etc. +// set up in advance; assigning `""` to `tr.innerHTML` doesn't work, etc. var wrapMapIE9 = { option: [1, ''], _default: [0, '', '']