%A_ThisMenuItem% needs to be fixed?

[eagerahk]

The document said,

To underline one of the letters in a menu item's name, precede that letter with an ampersand (&). When the menu is displayed, such an item can be selected by pressing the corresponding key on the keyboard.

The following code does exactly that, but the message coming out from %A_ThisMenuItem% is &MyItem1, My&Item2 and MyItem&3, rather than
MyItem1, MyItem2 and MyItem3. I don't think that's what we want. How can I fix this? Thanks.

Code: Select all

Menu, MyMenu, Add, &MyItem1, MenuHandler
Menu, MyMenu, Add, My&Item2, MenuHandler
Menu, MyMenu, Add, MyItem&3, MenuHandler
Menu, MyMenu, Show
return

MenuHandler:
MsgBox You selected "%A_ThisMenuItem%" from the menu %A_ThisMenu%.

[BoBo]

I don't think that's what we want

Well, I think that's what I want

Code: Select all

MenuHandler:
MsgBox % "You selected " StrReplace(A_ThisMenuItem,"&","") " from the menu " A_ThisMenu "."

Not tested.

[eagerahk]

Thanks for the reply, I just dont't know why one needs that & in &MyItem1, My&Item2 and MyItem&3? Do I miss anything here?

[BoBo]

Thanks for the reply, I just dont't know why one needs that & in &MyItem1, My&Item2 and MyItem&3? Do I miss anything here?

You've posted the excerpt from AHK's help that exactly explains that 'keyboard shortcut'-feature.
You won't have to use it if you don't use the keyboard to trigger menu-items.

Correct me f I'm wrong.

[eagerahk]

The value of %A_ThisMenuItem% does not need to carry that &.

[just me]

To underline one of the letters in a menu item's name, precede that letter with an ampersand (&). When the menu is displayed, such an item can be selected by pressing the corresponding key on the keyboard. To display a literal ampersand, specify two consecutive ampersands as in this example: Save && Exit

When referring to an existing menu or menu item, the name is not case sensitive but any ampersands must be included. For example: &Open

Source: Menu

[eagerahk]

Thanks, just me. If %A_ThisMenuItem% can drop one &.
Then in the case of My&Item2, the desired value of %A_ThisMenuItem% should be MyItem2.
In the case of Save && Exit, the desired value of %A_ThisMenuItem% should be Save & Exit.

So that each user does not have to do it.

[BoBo]

TBH, I don't understand why you get the idea that it's a problem at all, especially for those ~7.473.690.000 other people out there ("... that's what we want/each user ...")???

[just me]

When referring to an existing menu or menu item, the name is not case sensitive but any ampersands must be included. For example: &Open

That's also true if you refer to A_ThisMenuItem.

... , so that each user does not have to do it.

Why do you have to do it?

[Capn Odin]

Think of it this way, if you want to change something regarding a sub-menu you will use.

Code: Select all

Menu, MyMenu, Add, &AHK, MenuHandler
return

MenuHandler:
	Menu, MyMenu, Add, % A_ThisMenuItem, NewMenuHandler
return

NewMenuHandler:
	ExitApp
return

RButton::Menu, MyMenu, Show

Witch only works because the & are included.

Else you will get a new entry.

Code: Select all

Menu, MyMenu, Add, &AHK, NoAmpersand
return

NoAmpersand:
	Menu, MyMenu, Add, % StrReplace(A_ThisMenuItem, "&", ""), NewMenuHandler
return

NewMenuHandler:
	ExitApp
return

RButton::Menu, MyMenu, Show