Ok. So, keep doing your way for now. We can think about integrate than in the future depending on the viability after all stuff you will add.
But, since the Import/Export stuff you implemented act over the native Palette it would be appropriated those options be appended to the Import/Export option we already find there (Palette window), because it will not be intuitive to have two places to handle this kind of stuff.
I started to build a thread bout menus for programmers and at the end I hope this can help you spend more time coding than trying to figure out how the things work:
"How to" about context menus
I'm going to keep adding info there later.
But, since the Import/Export stuff you implemented act over the native Palette it would be appropriated those options be appended to the Import/Export option we already find there (Palette window), because it will not be intuitive to have two places to handle this kind of stuff.
edb Wrote:I can try adding the palette import/export stuff right away to the menu of wings_palette.erl, but I don't know how to make submenus using wings_menu:popup_menu yet. I first tried by doing a second psend/popup_menu call after the first psend/popup_menu in a handle_event. But nothing appears after the click but it does appear after a second right click, but I don't think I'm doing it right.You didn't asked for help, but I think you don't need to go through the hard path I've made.
I started to build a thread bout menus for programmers and at the end I hope this can help you spend more time coding than trying to figure out how the things work:
"How to" about context menus
I'm going to keep adding info there later.