Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0041155Openbravo ERPA. Platformpublic2019-06-24 13:022022-02-01 07:24
Reporteraaroncalero 
Assigned ToTriage Platform Base 
PrioritynormalSeverityminorReproducibilityalways
StatusnewResolutionopen 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerGold
Web browserGoogle Chrome
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0041155: Mouse scrolling generates errors in the browser console in chrome 73+
DescriptionWhen using Openbravo backend in a Chrome browser (version 73 or up), scrolling anywhere in the application (quick launch, windows in grid/form view) shows the following error in the browser console:
ISC_Combined.js:1402 [Intervention] Unable to preventDefault inside passive event listener due to target being treated as passive. See https://www.chromestatus.com/features/6662647093133312 [^]
Steps To ReproduceLog in in openbravo backend: https://livebuilds.openbravo.com/erp_pi_pgsql/ [^]
Open the browser console.
In the application, click on the Quick Launch button on the top left corner of the screen.
Display the Quick Lauch combo list.
Scroll up/down the list using the mouse wheel or laptop touchpad.
Verify that the error appears multiple times on the browser console.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0041664pi closed AugustoMauch Smartclient behaves wrongly when creating a record with a filled filter 
Attached Filesdiff issue41155.diff (1,161) 2019-06-24 15:49
https://issues-new.openbravo.com/file_download.php?file_id=13049&type=bug
Issue History
Date ModifiedUsernameFieldChange
2019-06-24 13:02aaroncaleroNew Issue
2019-06-24 13:02aaroncaleroAssigned To => platform
2019-06-24 13:02aaroncaleroOBNetwork customer => No
2019-06-24 13:02aaroncaleroWeb browser => Google Chrome
2019-06-24 13:02aaroncaleroModules => Core
2019-06-24 13:02aaroncaleroTriggers an Emergency Pack => No
2019-06-24 13:17AugustoMauchNote Added: 0112976
2019-06-24 13:18AugustoMauchNote Edited: 0112976bug_revision_view_page.php?bugnote_id=0112976#r18979
2019-06-24 15:49AugustoMauchNote Added: 0112978
2019-06-24 15:49AugustoMauchFile Added: issue41155.diff
2019-06-24 16:05guillermogilOBNetwork customerNo => Gold
2019-06-24 16:05guillermogilWeb browserGoogle Chrome => Google Chrome
2019-08-23 18:25caristuRelationship addedrelated to 0041664
2019-10-18 15:33rafarodaIssue Monitored: rafaroda
2022-02-01 07:24alostaleAssigned Toplatform => Triage Platform Base

Notes
(0112976)
AugustoMauch   
2019-06-24 13:17   
(edited on: 2019-06-24 13:18)
mouseWheel (Canvas.js:15578)
invokeSuper (Class.js:1550)
Super (Class.js:1368)
mouseWheel (ListGrid.js:990)
bubbleEvent (EventHandler.js:4751)
handleMouseWheel (EventHandler.js:5742)
dispatch (EventHandler.js:6152)
(anonymous) (VM95:3)

mouseWheel : function () {
    if ((this.overflow == isc.Canvas.AUTO || this.overflow == isc.Canvas.SCROLL) &&
        this.showCustomScrollbars && this.vscrollOn)
    {
        var wheelDeltaY = this.ns.EH.lastEvent.wheelDeltaY,
            wheelDeltaX = this.ns.EH.lastEvent.wheelDeltaX;
        // For each increment the user scrolled the mouse wheel, we want to move about 50px
        // This seems to approximately match native scrolling speed.
        var scrollTop =
                this.scrollTop + Math.round(wheelDeltaY * isc.Canvas.scrollWheelDelta);
        var scrollLeft =
                this.scrollLeft + Math.round(wheelDeltaX * isc.Canvas.scrollWheelDelta);

        // Note that scrollTo already catches scrolling past beginning or end
        this.scrollTo(scrollLeft, scrollTop, "mouseWheel");

        // return false to cancel further / native processing
        return false;
    }

    // Not a scrollable region, return true
    return true;
},
(0112978)
AugustoMauch   
2019-06-24 15:49   
A patch has been attached. It fixes the problem that its secondary effects need to be thoroughly resesarched.