X Tutup
Skip to content

Add better clarification for accelerator support on Popup Menu#113197

Merged
akien-mga merged 1 commit intogodotengine:masterfrom
CrayolaEater:pop-up-menu-doc-update
Dec 1, 2025
Merged

Add better clarification for accelerator support on Popup Menu#113197
akien-mga merged 1 commit intogodotengine:masterfrom
CrayolaEater:pop-up-menu-doc-update

Conversation

@CrayolaEater
Copy link
Contributor

@CrayolaEater CrayolaEater requested a review from a team as a code owner November 26, 2025 16:41
@Mickeon
Copy link
Member

Mickeon commented Nov 26, 2025

The parentheses at the end of the sentence imply that both pressing an item and activating an accelerator cause the signal to be emitted.

@CrayolaEater
Copy link
Contributor Author

The parentheses at the end of the sentence imply that both pressing an item and activating an accelerator cause the signal to be emitted.

I've changed the phrasing.

@CrayolaEater CrayolaEater force-pushed the pop-up-menu-doc-update branch from 96dd082 to 6b7ec99 Compare November 27, 2025 14:11
@Mickeon Mickeon modified the milestones: 4.x, 4.6 Nov 27, 2025
@akien-mga akien-mga merged commit 8f14acb into godotengine:master Dec 1, 2025
39 of 40 checks passed
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PopupMenu MenuButton accelerators don't emit id_pressed signal.

3 participants

X Tutup