Page 1 of 1

Menu accelerators in popupmenu

Posted: 24 Feb 2022 11:15
by MBaas
After the recent round of changes, popupmenu no longer handles accelerators in menus (and shows the "&" character instead):
Is that intentional (and gone forever) or unintended?

Repro (with 22.90.0011):
text popupnested("&Top| Sub| Sub| Sub2| Sub2|T&op");

Re: Menu aceelerators in popupmenu

Posted: 24 Feb 2022 11:18
by highend
and gone forever
I hope not^^ It breaks the navigation...

In my case, n doesn't work anymore to trigger the "New" menu part
2.png

Re: Menu accelerators in popupmenu

Posted: 24 Feb 2022 13:04
by admin
Not intended. :)

Re: Menu accelerators in popupmenu

Posted: 24 Feb 2022 13:12
by MBaas
:D Good to know

Re: Menu accelerators in popupmenu

Posted: 25 Feb 2022 13:42
by highend
Fixed in v22.90.0012

Re: Menu accelerators in popupmenu

Posted: 25 Feb 2022 14:14
by MBaas
[x] Confirmed :)

Re: Menu accelerators in popupmenu

Posted: 14 Jan 2024 12:40
by MBaas
I'm sorry to say- but suddenly this became an issue again. Not generally, though - it most often works fine - but I have one case where it does not work in a submenu. Fortunately I've been able to build a small repro. Please use the attached sample and then do

::popupnested(readfile("{yourPath}underline.txt"));

Alternatively, here's the content of that menu file:

Code: Select all

      &menu;:SwitchProjectMenu("menu");:refresh
      &coffee;:SwitchProjectMenu("coffee");:coffee
      &Jenkins;:SwitchProjectMenu("jenkins");:fore
      &TamStat;:SwitchProjectMenu("tamstat");:ghost
      &Vue;:SwitchProjectMenu("vue");:conf
P.S: using version 25.40.0305

Re: Menu accelerators in popupmenu

Posted: 14 Jan 2024 17:45
by admin
Confirmed. Fix comes.

Re: Menu accelerators in popupmenu

Posted: 15 Jan 2024 11:39
by MBaas
Fix works fine with all of my menus! Thanks! :cup: