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

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.

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

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

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.

@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.

Yeah these 2 changes are fine.

Thanks!