buberdds / angular-bootstrap-colorpicker

Native AngularJS colorpicker directive. No dependency on jQuery or jQuery plugin is required.
MIT License
419 stars 226 forks source link

Pointer is wrong when document is scrolled on Chrome #14

Closed leoperria closed 11 years ago

leoperria commented 11 years ago

Test scenario to reproduce the bug:

1) use Chrome browser 30.0.1599.69 (but I think is not version dependent) 2) scroll the page a bit 3) try the color picker

I made a quick patch that make everything ok on Chrome altering sliderUpdate:

I replaced: pointer = { left: event.pageX, top: event.pageY };

with

pointer = { left: event.clientX, top: event.clientY };

But it's only a workaround

buberdds commented 11 years ago

Thanks for your description and hints where should I look for a bug.