search

1j01 / jspaint

🎨 Classic MS Paint, ＲＥＶＩＶＥＤ + ✨Extras
https://jspaint.app/about
MIT License
7.18k stars 560 forks source link

Multi level menus does not work as expected. #208

Closed ippezshelby closed 3 years ago

captbaritone commented 3 years ago

They seem fine to me. Could you include an explanation of what seems broken about them? Maybe a screenshot?

ippezshelby commented 3 years ago

This is a 3 level menu. 1st and 2nd work fine but when you hover to the third level menu, the parent menu (2nd level) disappears leaving a blank gap between first level and third level as in the screenshot attached.

image Console log screenshot image

1j01 commented 3 years ago

It looks like you're trying to create an entirely different application using the menu code. I actually have that separately distributed as a library (and with a proper license) here: https://github.com/1j01/os-gui Feel free to open an issue there. (By the way, I want to rewrite the menu code eventually, as there's a lot of things I want to improve.)

ippezshelby commented 3 years ago

@1j01, any update on the menu code?

1j01 commented 3 years ago

@Ippezrobert Nope.

ippezshelby commented 3 years ago

@captbaritone , could you kindly share an example for this where it works fine or you could just make an adjustment on https://jspaint.app/ for me to check out. may be there is something am not doing right. thanks

1j01 commented 2 years ago

@Ippezrobert I've just released a new version of OS-GUI.js with this fixed, along with many other menu improvements. You can see the changelog for details. v0.4.0 is largely the same API, but I plan to make huge changes to the API in the next version. You can download v0.4.0 here. Let me know if you have any issues upgrading.

ippezshelby commented 2 years ago

@1j01, i will try it out.... how i wish this OS-GUI.js has most of the commonly used controls to develop a complete application like in Vb6... Examples of this include:

  1. DataGrid/Tables
  2. ListBox
  3. ComboBox
  4. Calendar/DateTime Picker
  5. MaskedTextBox