Results 1 to 3 of 3

Thread: Clicking a menu separator leads to a JS error

    Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha Premium Member valio's Avatar
    Join Date
    Jan 2014
    Posts
    203

    Default Clicking a menu separator leads to a JS error

    ExtJS Version:
    5.1.3

    Description:

    When clicking a menuseparator in a menu a JS error is thrown

    Expected Result:
    The click is ignore, no JS error is thrown.

    Fiddle with example:


    Workaround:
    As this bug is caused by a private method of Ext.menu.Menu, no good workaround is possible AFAIK (i.e. an override).
    One could theoretically open Ext.menu.Menu and replace the current onFocusableContainerMousedown method in the framework code
    Code:
    //mostly copy-paste from Ext.menu.Menu.onFocusableContainerMousedown in ExtJS 5.1.3
        onFocusableContainerMousedown: function(e, target) {
            var targetCmp = Ext.Component.fromElement(target);
    
    
            this.mousedownTimestamp = targetCmp === this ? Ext.Date.now() : 0;
    
    
            //only instances of the menuseparator widget (and its subclasses) has the separatorCls property
            if (targetCmp.separatorCls || targetCmp === this || !targetCmp.getFocusEl().contains(target)) {
                e.preventDefault();
            }
        }

  2. #2
    Sencha User
    Join Date
    Feb 2013
    Location
    California
    Posts
    11,985

    Default

    Thanks for the report. Looks like this issue has been resolved. Although I can see it in 5.1.3, I can't recreate using a nightly release of 5.1.4.

  3. #3

    Default

    Thanks valio, Your suggestion helped me to create an override. we were not in a state upgrade the Ext JS build.

Similar Threads

  1. [FIXED] Clicking on a context menu but not on a menu item raises error.
    By JimGrimmett in forum Ext 5: Bugs
    Replies: 2
    Last Post: 11 Feb 2015, 1:07 PM
  2. Replies: 0
    Last Post: 1 Apr 2012, 10:38 PM
  3. Replies: 0
    Last Post: 1 Apr 2012, 10:24 PM

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •