Add shortcuts in main buttons of MUI prefs

[Samir Hawamdeh]

Problem

The main buttons in MUI prefs lacks a proper shortcuts in order to select them via keyboard

I mean that one:

Save
Use
Test
Cancel

In english they might become:

_Save
_Use
_Test
_Cancel

Of course If this combination does not interferise with other shortcuts present

[Thore Böckelmann]

Although this sounds like a sensible request it might turn out to be not possible as easy as it seems. The point is that certain shortcut characters are already used otherwise. For english this work, but for example for german the translation for "Cancel" is "Abbrechen" and "A" is used for the "Save as" menu item already. I could use the "c" like in english, but I think you see the global problem to accomplish this task in all languages. For the italian translation we have the same problem with "Annulla". "n" and "l" are currently unused.

What makes the problem even worse are 3rd party .mcp modules. Usually such modules should not use any shortcuts for any of their labels/objects. MUI will try to distribute as many available shortcuts as possible automatically. But if a module does this itself already and uses "U" as shortcut for one of its own objects then MUI's own "Use" button will have a duplicate shortcut.

[Thore Böckelmann]

I just iterated over old tickets and also looked at the recently finished french translation. The french translation features shortcuts for the four bottom buttons of MUI prefs, although the built in english texts don't specify any shortcuts. There is absolutely no reason why a specific translation shouldn't provide any shortcuts when the built in string don't provide shortcuts. In the end it is the translator's task to provide suitable shortcuts anyway, no matter which shortcut (if any at all) is used for the built in strings.

To make it short: just go ahead and add shortcuts in the italian translation if you like. There is nothing that MUI needs to provide first of all.

[Samir Hawamdeh]

Mmm ok meanwhile i found a broken shortcut into the expired window, pls can you fix it

Grab attached

[Thore Böckelmann]

That's because MUI is falling back to EasyRequest(), because at the time MUI displays the expiry requester there is no MUI application object yet. And EasyRequest() of intuition.library does not support shortcuts.

That's what you get when you add shortcuts where the developer intentionally did not provide shortcuts.

I think I need to extend the EasyRequest() fallback to filter out such MUI specific things.

[Thore Böckelmann]

In 6320:

• masterreq.c: remove underscores and other MUI specific characters from the buttons string when falling back to EasyRequest(). This refs #330.

[Thore Böckelmann]

In 6321:

• masterreq.c, mastertext.c: use a plain text version of the body text passed to MUI_RequestA() when falling back to EasyRequest(). This refs #330.