desura / Desurium

Free online games platform (juegos gratis), with an open source client. LGPL repo for Desura client. Potentially out of date. See https://github.com/desura/desura-app for newest (LGPL) client.
https://www.desura.com/es
GNU General Public License v3.0
270 stars 42 forks source link

Added control + c (copy) to message dialogs to copy text #373 #682

Closed lodle closed 11 years ago

Jookia commented 11 years ago

Could we perhaps have this not as WXK_CONTROL_C but something like keys::copyKey() ? I'm just worried about in future if we have to abide to UX guidelines.

lodle commented 11 years ago

What is keys::copyKey()? i dont see why this is such a big issue. If it needs to change we can always revisit it later.

Jookia commented 11 years ago

It's a hypothetical thing. Are we sure in the future we won't be adding more keybindings?

lodle commented 11 years ago

If we did i think we would look into a solution that would be more generic than hard coded values. For now this is fine