Fix multiple shortcuts for actions

[melkorbsd]

No description provided.

[royqh1979]

Why not use QKeySequence::listFromString?

[melkorbsd]

QKeySequence::listFromString

There was such variant:

action->setShortcuts( QKeySequence::listFromString ( shortcut.replace (',' , ';' ) ) );

but it turned out to be inoperable.

[royqh1979]

There was such variant:

action->setShortcuts( QKeySequence::listFromString ( shortcut.replace (',' , ';' ) ) );

but it turned out to be inoperable.

1. What's the purpose of replacing ';' to ','? Why don't just use listToString() 's default output?
2. Why the output of listFromString is not workable？ It should return a list of qkeysequence.
3. Manually split string may break if the key squence contains 'ctrl+,' etc.

[melkorbsd]

There was such variant:
action->setShortcuts( QKeySequence::listFromString ( shortcut.replace (',' , ';' ) ) );
but it turned out to be inoperable.

1. What's the purpose of replacing ';' to ','? Why don't just use listToString() 's default output?

Because in shortcutmanager uses ',' as multiple shortcut delimiter ("Ctrl+C, Ctrl+Ins"), but listToString() return string value with ';' delimiters.

2. Why the output of listFromString is not workable？ It should return a list of qkeysequence.

3. Manually split string may break if the key squence contains 'ctrl+,' etc.

'Ctrl+,' cant, because ',' used as delimiter. Maybe change delimiter to ';' ?

[melkorbsd]

I change PR commits, new variant with listFromString and ';' delimiters.