submenu can be off screen about ctxmenu HOT 10 CLOSED

The problem still remains. I tried the fix #14 but it doesn't fix the problem

from ctxmenu.

The changes are not on master branch yet, make sure you download from the 1.2 branch:
https://raw.githubusercontent.com/nkappler/ctxmenu/1.2/standalone/ctxmenu.js

Also, if you use images in your menu (I guess you do with the icons)
Make sure they are preloaded to the cache. Otherwise the size calculation happens before the image load and the menu ends up larger than it thinks.

Close your devtools, if you use the "disable cache" option.

This should do the trick hopefully.
Making the size calculation handle uncached images is more work, which I postponed for now

from ctxmenu.

There are still problems. In the lower right part of the screen, if you open the menu 2 times in a row, a bug appears a second time. The first time goes well. Make 2 menu items and switch them one by one in this corner. But there is already progress

from ctxmenu.

Our layout designer said that it is very wrong to embed css settings in a js file. Some styles overlap each other. Therefore, you need to keep everything separate.

from ctxmenu.

thanks for the hint, I fixed the issue.

Having some css properties in the code is intentional, otherwise I would need to make another request and load a second file, which I don't want. I want to keep it simple and fast.
The generated css is added as the first style sheet to the DOM, so overriding is very easy, as the included css rules have the least possible priority. This is done so you can restyle it easily. There should be nothing overlapping, are you facing problems?

Possible Future themes will be handled as css files though, as they would increase the file size otherwise and they are not needed by everyone.

from ctxmenu.

Now there is another problem. It was not there before. The menu does not close after other actions start. When opened through the menu of a modal window, the context menu itself does not close and always remains on top. See the screen.

Or you need a method to close the menu.

from ctxmenu.

P.S. I found you have a new undocumented method that allows you to close the menu. It worked for me. Now I can close the menu before opening the modal. It is perfectly. Excellent result. The previous message is no longer relevant.
ctxmenu.closeMenu();

from ctxmenu.

closeMenu is a private method as you should not nees to do this... I thought i fixed it already 😅

from ctxmenu.

Yes, everything has already been fixed. I wrote this before it was fixed. Everything is fine

from ctxmenu.

ok, great to hear 👍

from ctxmenu.