Page 1 of 1
new popupmenu feature
Posted: 20 Feb 2022 01:45
by jupe
While testing the new popupmenu/nested feature of automatically going to paths and launching exe's, I noticed that it seems it will break scripts use of popupmenu when using paths, eg.
e popupmenu("<xydata>|<xypath>");
will take you to the path in addition to the echo, another new behavior is that it will also now automatically only show the folder/file name without path, but that is probz ok in most historic scripts. Files seem ok though, just the echo is returned in that case.
Re: new popupmenu feature
Posted: 20 Feb 2022 12:33
by admin
Fix on the way.
Re: new popupmenu feature
Posted: 21 Feb 2022 01:07
by jupe
Confirm fixed in v22.90.0004
Re: new popupmenu feature
Posted: 05 Mar 2022 05:08
by jupe
I noticed that the default icon for all menu items when used in popupmenu is now a hamburger style icon, is that intentional?
popupmenu("#302|#303|#304");
2022-03-05_150107.png
Re: new popupmenu feature
Posted: 05 Mar 2022 10:19
by admin
Actually yes. I forgot to mention it in the change log. The idea is that menu commands get the menu icon.
So, uhm, you don't like it?
Re: new popupmenu feature
Posted: 05 Mar 2022 10:27
by jupe
I am not fussy, just was unexpected.
Re: new popupmenu feature
Posted: 05 Mar 2022 10:32
by jupe
Actually, it blocks items that have a checked state from displaying it though, eg:
popupmenu("#480|#801");
Re: new popupmenu feature
Posted: 05 Mar 2022 10:42
by admin
Argh!

MS's age-old bug of having symbols and ticks in the same place will haunt us forever...
Re: new popupmenu feature
Posted: 05 Mar 2022 17:14
by jupe
While testing this I noticed that none of the scripting menu's items display their checked state in popupmenus/HB, unlike when shown in the main menu, not sure if that was already known, so thought I'd mention it, its like this in older versions too, so not a new issue.
popupmenu("#750|#758");
Additionally since we're on the subject, a fair few of the main submenu items don't show state, that may be by design though:
popupmenu("#681|#682|#687|#402|#404"); // 687 works though, so some submenu items do
that isn't a complete list, it was just an example of a few CIDs that don't.
Re: new popupmenu feature
Posted: 07 Mar 2022 09:10
by admin
A neglected area indeed. Will be better in the next beta.
Re: new popupmenu feature
Posted: 07 Mar 2022 17:26
by jupe
I noticed these still aren't working: popupmenu('#691|#692|#693|#694');
Re: new popupmenu feature
Posted: 07 Mar 2022 17:47
by admin
Thanks!
Re: new popupmenu feature
Posted: 07 Mar 2022 18:11
by jupe
You're going to hate me, but #702 #703 too.
Re: new popupmenu feature
Posted: 07 Mar 2022 19:07
by admin
Well I hope you don't hate me for letting you find all those bits instead of fixing the whole thing for good.