[GUI] remember last used keyboard layout

[da-anda]

When having multiple keyboard layouts selected, the OSD keyboard will always show the first layout in the list. With this change, Kodi will remember the last used keyboard layout and will use that instead.

Description

A hidden setting has been added which will be updated each time the layout is being changed.

Motivation and Context

There is currently no way to define a preferred keyboard layout. Even when you select the keyboard layouts in a specific order, they will always be stored in alphabetical order. Remembering the last used layout makes a custom order superflous and is even more flexible.

How Has This Been Tested?

runtime tested

Types of change

added hidden setting to store the last active keyboard layout

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

[Memphiz]

I think its fine as is. There is no other way to get the index as long as those are vectors and not dictionaries.

[da-anda]

thanks for review. I'll wait until sunday, and will merge then if nobody objects.

@MartijnKaijser trivial enough for a backport?

[phil65]

@Memphiz Wouldn't std::distance() work here ?
I think we should keep backports to critical fixes at this stage.

[MartijnKaijser]

ping @Montellese if he knows another way

[Memphiz]

@phil65 size() is already a shortcut to distance() as of the vector.size() docu...

[Memphiz]

Also there is nothing wrong with that way - the for loop is already there - size has constant complexity - i find no reason why this should be unclean...

[phil65]

sure. @Montellese pointed me to std::distance() some weeks ago for very similar use case, that´s why I was askin.
All fine then, feature makes sense.