From 143c61a5502fc3cb78ba12237cacf6b8bf836797 Mon Sep 17 00:00:00 2001 From: Sebastian Widmer Date: Tue, 11 Feb 2014 11:07:04 +0100 Subject: [PATCH 1/3] fix browser detection for jQuery > 2.0 jQuery removed $.support.leadingWhitespace from jquery greater 2.0 because support was dropped for IE 7/8. So if property is undefined, we are pretty shure its ie > 8 or jQuery does not support the browser anyway. --- src/jquery.flot.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/jquery.flot.js b/src/jquery.flot.js index 72bb07e..0c5a985 100644 --- a/src/jquery.flot.js +++ b/src/jquery.flot.js @@ -361,7 +361,7 @@ Licensed under the MIT license. c = sin.toFixed(6), // when we add them to the string transformRule; - if ($.support.leadingWhitespace) { + if (!($.support.leadingWhitespace === false)) { // The transform origin defaults to '50% 50%', producing // blurry text on some browsers (Chrome) when the width or From 6fd170d03423bb541bfefcec4e0f99a857187a7e Mon Sep 17 00:00:00 2001 From: Sebastian Widmer Date: Tue, 11 Feb 2014 11:26:29 +0100 Subject: [PATCH 2/3] no confusing use of "!" --- src/jquery.flot.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/jquery.flot.js b/src/jquery.flot.js index 0c5a985..b27886f 100644 --- a/src/jquery.flot.js +++ b/src/jquery.flot.js @@ -361,7 +361,7 @@ Licensed under the MIT license. c = sin.toFixed(6), // when we add them to the string transformRule; - if (!($.support.leadingWhitespace === false)) { + if ($.support.leadingWhitespace !== false) { // The transform origin defaults to '50% 50%', producing // blurry text on some browsers (Chrome) when the width or From 5f59ec87c9260a08a17bfb632b2894832b08cd5f Mon Sep 17 00:00:00 2001 From: Sebastian Widmer Date: Wed, 12 Feb 2014 08:50:12 +0100 Subject: [PATCH 3/3] add comment to browser detection --- src/jquery.flot.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/jquery.flot.js b/src/jquery.flot.js index b27886f..f3f9023 100644 --- a/src/jquery.flot.js +++ b/src/jquery.flot.js @@ -361,6 +361,9 @@ Licensed under the MIT license. c = sin.toFixed(6), // when we add them to the string transformRule; + // Detect IE7/8 to use microsofts proprietary matrix filter + // $.support.leadingWhitespace is false on IE7/8, true on other browsers and + // undefined with jQuery greater 2.0, which dropped support for IE7/8. if ($.support.leadingWhitespace !== false) { // The transform origin defaults to '50% 50%', producing