Ext.Menu swipe does not work, even in kitchen sink

[pincher]

We are using touch 2.3.1 and tried to implement the Ex.Menu.

Not with your example kitchen sink nor using the Ext.Menu in our application the swipe (from whatever edge) works, toggleMenu has no problem. Tried on iPad (iOS7, latest version Safari/Chrome), OSX-Mavericks (latest version Safari/Chrome)

The errors shown on the console in all cases (Kitchen sink iPad, desktop) are:

- TypeError: 'undefined' is not an object (evaluating 'this.getTranslatable().on')
>>> Ext.viewport.Default#onEdgeSwipeEnd

...
if (menu.$cover) {
            menu.getTranslatable().on('animationend', function() {
                if (shouldRevert) {
                    menu.hide();
                }
            }, this, {
                single: true
            });


            menu.translate(after.translateX, after.translateY, {
                preserveEndState: true,
                duration: 200
            });


        }
        else {
            this.getTranslatable().on('animationend', function() {
                if (shouldRevert) {
                    menu.hide();
                }
            }, this, {
                single: true
            });


            this.translate(viewportAfter.translateX, viewportAfter.translateY, {
                preserveEndState: true,
                duration: 200
            });
        }

and
- TypeError: 'null' is not an object (evaluating 'point.x')
>>> Ext.util.Point#getDistanceTo

[mitchellsimoens]

Thanks for the report! I have opened a bug in our bug tracker.

[lanbo]

I'm also experiencing the same bug. It's strange after so many months the bug is still around. No one uses Ext.Menu component? I thought this one would be extremely popular but there are not many complaints though.
Any plans to fix it or any workaround? Thanks!

[vitalii.blagodir]

My workaround for Sencha Touch 2.4.1 is to replace in touch\src\viewport\Default.js line 1043

if (checkMenu.isHidden() !== false) {

with

if (checkMenu.isHidden() == false) {

See also here
http://stackoverflow.com/questions/2...93950#29093950

[Tarabass]

That would be

if (checkMenu.isHidden() === false) {

don't you think

[mtkutz]

My workaround for Sencha Touch 2.4.1 is to replace in touch\src\viewport\Default.js line 1043

if (checkMenu.isHidden() !== false) {

with

if (checkMenu.isHidden() == false) {

See also here
http://stackoverflow.com/questions/2...93950#29093950

This worked for me in Touch 2.4.1 as well.

Instead of modifying the Touch source, I created an override.

[CaCtus491]

This bug still exists in ExtJS6 modern toolkit. The same fix still applies.

[CaCtus491]

FYI, the override we are using for ExtJS6 Modern is below. We were getting errors in onEdgeSwipeEnd too.

In our application, we don't want to be able swipe from the side to bring the menu up (must click the button instead) as we have a carousel which the menu gets in the way of when trying to swipe through items, so we can get away with short-circuiting things.

Ext.define('App.override.viewport.Default', {
    override: 'Ext.viewport.Default',

    onSwipeStart: function(e) {
    	e.stopPropagation();
    },
    
    onEdgeSwipeStart: function(e) {
    	e.stopPropagation();
    	return;
    },
    
    onEdgeSwipeEnd: function(e) {
    	e.stopPropagation();
    	return;
    }
});

[pier77]

I have the same problem with Sencha 6.0.1 The line should be 1033

if (checkMenu.isHidden() == false) { return; }

[Daniil]

It would be nice to see it eventually fixed in ExtJS 6 modern toolkit. Dear colleagues, thank you for reporting this bug and workaround. It helped a lot.