Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[keyboard layouts] use English language names #5525

Merged
merged 1 commit into from
Oct 18, 2014

Conversation

un1versal
Copy link
Contributor

This normalizes language names to English and specify English in QWERTY/ABC/AZERTY

_Future consideration_
In future because the OSK Button label is rather limited in length that a _ID_ field could be implemented So that _layout name_ would be displayed in selection list and in _id_ would show in button label something like.

<layout name="English ABC" id=eng abc>
<layout name="English QWERTY" id=eng qwerty>

However that last part (implementing the id) I would require assistance in the C++ portion @stefansaraev understands this idea.

@MartijnKaijser hope this helps the author of #5419 make up his mind and rebase/change his PR.

Also added small comment to guide future contributors.
@topfs2 topfs2 added Helix Type: Fix non-breaking change which fixes an issue and removed Approved labels Oct 17, 2014
@topfs2
Copy link
Contributor

topfs2 commented Oct 17, 2014

@MartijnKaijser any thoughts? looks ok by me. This is the way we want it named no?

MartijnKaijser added a commit that referenced this pull request Oct 18, 2014
[keyboard layouts] use English language names
@MartijnKaijser MartijnKaijser merged commit aed061b into xbmc:master Oct 18, 2014
@MartijnKaijser MartijnKaijser added this to the Helix 14.0-alpha5 milestone Oct 18, 2014
un1versal pushed a commit to un1versal/kodi that referenced this pull request Oct 20, 2014
This fixes to match xbmc#5525 setting name is now English QWERTY
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Fix non-breaking change which fixes an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants