helloSystem / QKeySequenceOverride

Override the QKeySequence::toString() function in Qt to represent keyboard shortcuts with graphical symbols
2 stars 2 forks source link

Consider showing "Alt" instead of "Command" for PC keyboards #2

Open probonopd opened 3 years ago

probonopd commented 3 years ago

Was considering showing "Alt" instead of "Command" for PC keyboards, but wasn't sure whether it would further complicate things (think screenshots and documentation).

Now it has been asked on Twitter: https://twitter.com/mister_lemke/status/1449008251622748163

Let's think about the best way to go about this.

daniel-lemke commented 3 years ago

I didn't think about extra documentation burden. Since helloSystem seems oriented towards converting Mac users, it may not be worth the added effort.

I just checked and the documentation does mention the correct key for PC keyboards. Displaying the correct symbol for the user's keyboard on system would be nice, but I doubt it should be considered a priority unless enough PC users speak up. I am willing to volunteer some development time on a solution.

probonopd commented 3 years ago

Technically this wouldn't be hard as we are already checking for Apple keyboards in other places but yes, let's see whether this will be frequently requested.