Thanks, the changes here look good.
I'm not sure if the KeyCombination suggestion is going to make things simpler? I think all we need is just a variant of CtrlLabel () to produce the label for a shortcut, which seems pretty straightforward

You are right @cameronwhite . I am not that acquainted with Gtk, so I was not aware of what Gtk.Functions.AcceleratorGetLabel did until I looked at CtrlLabel.

I could do what you suggest. The main issue is getting hold of a SystemManager (for deciding if we want <Meta> or <Control>), which unfortunately requires changes in many files to do properly, but I could introduce a PintaCore reference and get rid of it in a future pull request. Or I could leave it as <Primary> for now. Update: see below.

I just did what you suggested @cameronwhite. The function Gtk.Functions.AcceleratorGetLabel shows the key combinations in a readable way, and it converts the <Primary> key to the right one for the OS in question (at least from what I could see by testing my changes on Ubuntu, namely, showing the primary key as 'Ctrl').

I think we do need translate the <Primary> modifier to have it work on macOS, otherwise it shows up as Ctrl when it should be Cmd

SystemManager.GetOperatingSystem() is static though so I don't think that's a big problem to introduce the reference

Thanks @cameronwhite for testing it on your machine. I've now applied your suggestion. Could you test again and confirm if it works?

Yep, that looks good now on Mac!