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

Uzbek layout #828

Merged
merged 6 commits into from
Dec 29, 2024
Merged

Uzbek layout #828

merged 6 commits into from
Dec 29, 2024

Conversation

tenextractor
Copy link
Contributor

From Gboard
image

Copy link
Owner

@Julow Julow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The layout is 11 columns, which is quite large for some devices. Couldn't oʻ and gʻ be moved in the corner of an other key to make all the keys bigger ?

@tenextractor
Copy link
Contributor Author

It's a copy of AOSP and Gboard layout, and doesn't change what people are used to.

But if needed we can change it to this:

q w e r t y u i o p
a s d f g h j k l ʻ (U+02BB turned comma)
z x c v b n m ʼ (U+02BC modifier letter apostrophe)

@Julow
Copy link
Owner

Julow commented Dec 19, 2024

Less wide would definitely be better.

@Julow Julow added this to the 1.31.0 milestone Dec 21, 2024
@tenextractor
Copy link
Contributor Author

image

Copy link
Owner

@Julow Julow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Let's merge.

@Julow Julow merged commit e594915 into Julow:master Dec 29, 2024
4 checks passed
Copy link
Owner

@Julow Julow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many thanks for your great contributions. You are based on an old commit. Could you make sure to pull the master branch before creating new branches in the future ?

@@ -58,5 +58,6 @@
<subtype android:label="%s" android:languageTag="tly" android:imeSubtypeLocale="tly_IR" android:imeSubtypeMode="keyboard" android:isAsciiCapable="true" android:imeSubtypeExtraValue="default_layout=arab_hamvaj_tly"/>
<subtype android:label="%s" android:languageTag="tr" android:imeSubtypeLocale="tr_TR" android:imeSubtypeMode="keyboard" android:isAsciiCapable="true" android:imeSubtypeExtraValue="script=latin,default_layout=latn_qwerty_tr,extra_keys=accent_cedille:ç:ş@c|accent_trema:ö:ü@u|accent_circonflexe:â:î:û@f|₺|ı|ğ"/>
<subtype android:label="%s" android:languageTag="uk" android:imeSubtypeLocale="uk_UA" android:imeSubtypeMode="keyboard" android:isAsciiCapable="true" android:imeSubtypeExtraValue="script=cyrillic,default_layout=cyrl_jcuken_uk,extra_keys=ґ|є|і|ї|₴"/>
<subtype android:label="%s" android:languageTag="uz" android:imeSubtypeLocale="uz_UZ" android:imeSubtypeMode="keyboard" android:isAsciiCapable="true" android:imeSubtypeExtraValue="script=latin,default_layout=latn_qwerty_uz,extra_keys=ʻ|ʼ"/>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've updated this file so that the layout is shown by default to Uzbek users. The extra_keys attribute lists the keys that should be automatically added on other latin layouts so that you can still type Uzbek on it.

Do you think you could update this file the next time you add a layout ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I will add it from the next time

@@ -61,6 +61,7 @@
<item>latn_qwerty_sr</item>
<item>latn_qwerty_tly</item>
<item>latn_qwerty_tr</item>
<item>latn_qwerty_uz</item>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is updated with gradle test. Do you think you could do that next time ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay

@Julow Julow mentioned this pull request Dec 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants