diff options
| author | Jose Antonio Marquez | 2012-06-25 15:07:44 -0700 |
|---|---|---|
| committer | Jose Antonio Marquez | 2012-06-25 15:07:44 -0700 |
| commit | a86fe8c4939e30fe20e2058136b3e44e835ae32d (patch) | |
| tree | 02eafc4ceca3b9ed7195ffdb5092b8ae3e3bc69f /js | |
| parent | 5f96077a9b2f8386aa2c024f52ad03e88dcb7ec7 (diff) | |
| download | ninja-a86fe8c4939e30fe20e2058136b3e44e835ae32d.tar.gz | |
Fixing bottom hit test for popup
This address clicking outside (below) and closing the popup. The wrong value was being used.
Diffstat (limited to 'js')
| -rwxr-xr-x | js/panels/color/colorpopup-manager.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/js/panels/color/colorpopup-manager.js b/js/panels/color/colorpopup-manager.js index cf6fd7ac..dc50154c 100755 --- a/js/panels/color/colorpopup-manager.js +++ b/js/panels/color/colorpopup-manager.js | |||
| @@ -114,7 +114,7 @@ exports.ColorPopupManager = Montage.create(Component, { | |||
| 114 | right = left + parseInt(element.popup.element.offsetWidth); | 114 | right = left + parseInt(element.popup.element.offsetWidth); |
| 115 | //Getting vertical limits | 115 | //Getting vertical limits |
| 116 | top = parseInt(element.popup.element.style.top) + parseInt(element.popup.element.style.marginTop); | 116 | top = parseInt(element.popup.element.style.top) + parseInt(element.popup.element.style.marginTop); |
| 117 | bottom = left + parseInt(element.popup.element.offsetHeight); | 117 | bottom = top + parseInt(element.popup.element.offsetHeight); |
| 118 | //Checking click position in relation to limits | 118 | //Checking click position in relation to limits |
| 119 | if ((e._event.clientX < left || e._event.clientX > right) || (e._event.clientY < top || e._event.clientY > bottom)) { | 119 | if ((e._event.clientX < left || e._event.clientX > right) || (e._event.clientY < top || e._event.clientY > bottom)) { |
| 120 | //Hides popups since click is outside limits | 120 | //Hides popups since click is outside limits |