Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

contextMenu submenus not working in Tablet #17

Closed
caubia opened this issue Jun 3, 2015 · 4 comments
Closed

contextMenu submenus not working in Tablet #17

caubia opened this issue Jun 3, 2015 · 4 comments

Comments

@caubia
Copy link

caubia commented Jun 3, 2015

Hi,

When you're on a tablet not trigger submenus. Would be able to do with a click.¿ Any way to fix this problem?

Thanks for advance.

Carles.

@s-yadav
Copy link
Owner

s-yadav commented Jun 3, 2015

Hi, there some work need to be done for fixing touch events properly,
For workaround you can change line 724

         subMenu: function (trigger, selector, baseTrigger, option) {
            trigger.contextMenu('menu', selector, {
                triggerOn: 'hover',
                displayAround: 'trigger',
                position: 'auto',
                baseTrigger: baseTrigger,
                containment: option.containment
            });
        },

to

         subMenu: function (trigger, selector, baseTrigger, option) {
            trigger.contextMenu('menu', selector, {
                triggerOn: 'mousenter touchend',
                displayAround: 'trigger',
                position: 'auto',
                baseTrigger: baseTrigger,
                containment: option.containment
            });
        },

Let me know if this solves your problem . Will be having proper fix soon.

@s-yadav
Copy link
Owner

s-yadav commented Jun 16, 2015

Hi caubia, committed v 1.4.0 with touch device support, make sure to update css file also.
let me know if you find any issue with latest build.

@caubia
Copy link
Author

caubia commented Jun 17, 2015

Sudhanshu,

I tried from the http://ignitersworld.com/lab/contextMenu.html#subMenus page and now works correctly with a smartphone. This afternoon I will try from a tablet. This is great :-).

Just one question, if I want to change the width / height size menu to adapt it to a smartphone, I should change some css rule?

Thanks !!!

Carles.

@s-yadav
Copy link
Owner

s-yadav commented Jun 17, 2015

Yes, you should write media queries, just be careful about overflow property, keeping overflow auto/hidden may hide submenus.

@s-yadav s-yadav closed this as completed Jul 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants