Skip to content

Remove the need to have a display name for built-in action overrides - #107487

Merged
Repiteo merged 1 commit into
godotengine:masterfrom
pippenpaddleopsicopolis:fix-shortcut-names
Oct 22, 2025
Merged

Remove the need to have a display name for built-in action overrides#107487
Repiteo merged 1 commit into
godotengine:masterfrom
pippenpaddleopsicopolis:fix-shortcut-names

Conversation

@pippenpaddleopsicopolis

@pippenpaddleopsicopolis pippenpaddleopsicopolis commented Jun 13, 2025

Copy link
Copy Markdown
Contributor

This PR

  • Adds missing built-in action display names
  • Fixes extra space in "ui_swap_input_direction "
  • Removes the need to add a display name for built-in action overrides

Comment thread core/input/input_map.cpp Outdated
@KoBeWi

KoBeWi commented Jul 28, 2025

Copy link
Copy Markdown
Member

The code looks good, but where are these names displayed? 🤔

@Mickeon

Mickeon commented Jul 29, 2025

Copy link
Copy Markdown
Member

The new strings added in this PR are not referenced anywhere, directly. I tried justifying their addition but I can't really find a reason as to why they're here. It's seemingly arbitrary.

@pippenpaddleopsicopolis

pippenpaddleopsicopolis commented Jul 29, 2025

Copy link
Copy Markdown
Contributor Author

The new strings added in this PR are not referenced anywhere, directly.

They are referenced in the code like any other built-in with an existing description. Take for example ui_text_completion_query which already has a description vs ui_text_completion_accept and ui_text_completion_replace for which the description is added in this PR. Is ui_text_completion_query not referenced as well?

I tried justifying their addition but I can't really find a reason as to why they're here. It's seemingly arbitrary.

What I did is making sure we have a description for all builtin actions. The goal was consistency rather than an arbitrary change.

The code looks good, but where are these names displayed? 🤔

That's a good question, and after checking I didn't find any usage in the editor. If the description is not used then maybe we should get rid of it all together. But then it seems we are still maintaining that list #107471 #87883 #76829

@KoBeWi

KoBeWi commented Jul 29, 2025

Copy link
Copy Markdown
Member

Some action names are displayed in context menus in LineEdit/TextEdit.

@pippenpaddleopsicopolis

Copy link
Copy Markdown
Contributor Author

Also a good place to have the description of the built-in shown is the Input Map tab in the project settings (on hover for example). But that needs a PR.
image

@pippenpaddleopsicopolis

Copy link
Copy Markdown
Contributor Author

@KoBeWi @Mickeon Do you think these two changes are still worth it?

  • Fixes extra space in "ui_swap_input_direction "
  • Removes the need to add a display name for built-in action overrides

If so I can remove the added strings.

@KoBeWi

KoBeWi commented Jul 29, 2025

Copy link
Copy Markdown
Member

Yeah these 2 changes are fine.

@pippenpaddleopsicopolis pippenpaddleopsicopolis changed the title Add missing built-in action display names in InputMap Removes the need to have a display name for built-in action overrides Jul 29, 2025
@AThousandShips AThousandShips changed the title Removes the need to have a display name for built-in action overrides Remove the need to have a display name for built-in action overrides Jul 29, 2025
@Repiteo Repiteo modified the milestones: 4.5, 4.6 Jul 29, 2025
@pippenpaddleopsicopolis

This comment was marked as off-topic.

@pippenpaddleopsicopolis

This comment was marked as off-topic.

@Calinou Calinou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good to me.

@Repiteo
Repiteo merged commit 9cd297b into godotengine:master Oct 22, 2025
20 checks passed
@Repiteo

Repiteo commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

Thanks!

BendyLand pushed a commit to BendyLand/voltaire that referenced this pull request Aug 2, 2026
…ames

Remove the need to have a display name for built-in action overrides
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.

6 participants