Skip to content

Conversation

@allejok96
Copy link
Contributor

This PR removes an unnecessary string argument from automatable buttons, resulting in shorter code and preventing meaningless translation work.

When I translated I discovered some strings that are not visible in the UI. It turns out that AutomatableButton, PixmapButton and LedCheckBox all require a name for their default constructed model. The string is displayed in the header of the right click context menu, but only if the button never gets connected to a real model. If that happens, the real model's name is used instead. 95% of buttons are connected to real models, and the 5% that aren't are probably not meant to be automated (eg not meant to be right clicked), so removing a header text from their context menu should be no big deal. You can see an example of this in the video. All button tooltips and labels remain untouched.

Skarminspelning.fran.2025-12-23.00-33-13.mp4

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant