From 9b004c41f37be6f3a6ad1542ae78759efec659ed Mon Sep 17 00:00:00 2001 From: calebroseland Date: Mon, 1 Feb 2016 09:37:37 -0500 Subject: [PATCH] Update jsColor.js use getBoundingClientRect instead of element offset to account for absolute positioned elements. --- javascript_implementation/jsColor.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/javascript_implementation/jsColor.js b/javascript_implementation/jsColor.js index fbbecac..44c5fe3 100644 --- a/javascript_implementation/jsColor.js +++ b/javascript_implementation/jsColor.js @@ -98,7 +98,8 @@ var onOff = off ? 'removeEventListener' : 'addEventListener', focusListener = function(e) { var input = this, - position = {left: input.offsetLeft, top: input.offsetTop}, + inputRect = input.getBoundingClientRect(), + position = {left: inputRect.left, top: inputRect.top}, index = multiple ? Array.prototype.indexOf.call(elms, this) : 0, colorPicker = colorPickers[index] || (colorPickers[index] = createInstance(this, config)), @@ -226,4 +227,4 @@ (options.secure ? '; secure' : ''); } }; -})(this); \ No newline at end of file +})(this);