wx accelerator separator is currently "-", not "+"

[troelsk]

Patch:
Replace "Ctrl+" with "Ctrl-"

Topic for another patch:
Make the default wx accelerator separator "+"; it is the default everywhere as far as I know

[troelsk]

2.8 branch

[troelsk]

Trunk

[vadz]

Thanks, will apply soon.

I guess we should indeed use '+' instead of '-' as I can't find any applications using the latter any more, I didn't even notice when this changed...

[VZ]

(In [62791]) Use wxAcceleratorEntry::ToString() for undo/redo accelerator labels.

Don't hardcode the string representation of the accelerators, it was
inconsistent with the rest of wxWidgets which used '-' while here a '+' was
used.

Closes #11512.

[troelsk]

- r62791: I think \t must stay.
- Ticket has patch for stable 2.8 branch also, safe to apply as it is tested and all

[vadz]

Replying to troelsk:

- r62791: I think \t must stay.

Oops, sorry, thanks for noticing this!

- Ticket has patch for stable 2.8 branch also, safe to apply as it is tested and all

Sorry, I don't want to change this in 2.8. It's hardly a fatal bug and you never know what can be broken by this (maybe someone relies on it being '+' in their code, I've seen weirder things happen).

[VZ]

(In [62794]) Restore TAB in accelerator string wrong removed by r62791.

See #11512.

[troelsk]

Sorry, I don't want to change this in 2.8. It's hardly a fatal bug...

Okay, I'll patch my local wx copy then, already a hackorama extraordinaire, sigh :-(
Or deal with it in user space. Naturally I would have preferred both patches to be applied, to both branches, with no tinkering...

[troelsk]

'+' is now the wx default, in trunk, since r64157