Sure thing, I'll make the change and test it.


On 31 July 2013 15:46, Tait Brown <notifications@github.com> wrote:

> In js/jquery.fn.gantt.js:
>
> > @@ -338,19 +341,38 @@
> >              dataPanel: function (element, width) {
> >                  var dataPanel = $('<div class="dataPanel" style="width: ' + width + 'px;"/>');
> >
> > -                // Handle mousewheel events for scrolling the data panel
> > -                var mousewheelevt = (/Firefox/i.test(navigator.userAgent)) ? "DOMMouseScroll" : "mousewheel";
> > -                if (document.attachEvent) {
> > -                    element.attachEvent("on" + mousewheelevt, function (e) { core.wheelScroll(element, e); });
> > -                } else if (document.addEventListener) {
> > -                    element.addEventListener(mousewheelevt, function (e) { core.wheelScroll(element, e); }, false);
> > +                if (settings.scrollOnWheel == true) {
> > +                    // Handle mousewheel events for scrolling the data panel
> > +                    var mousewheelevt = (/Firefox/i.test(navigator.userAgent)) ? "DOMMouseScroll" : "mousewheel";
>
> Considering this project has a jQuery dependency can we just use $(element).on("mousewheel",function()
> ..., as that would handle the event fallbacks?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/taitems/jQuery.Gantt/pull/83/files#r5496245>
> .
>