-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Added Gujarati language layout #27
base: main
Are you sure you want to change the base?
Conversation
@@ -204,6 +205,7 @@ fun Context.getKeyboardLanguageText(language: Int): String { | |||
LANGUAGE_FRENCH_BEPO -> "${getString(R.string.translation_french)} (BEPO)" | |||
LANGUAGE_GERMAN -> getString(R.string.translation_german) | |||
LANGUAGE_GREEK -> getString(R.string.translation_greek) | |||
LANGUAGE_GUJARATI -> getString(R.string.translation_gujarati) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This string doesn't exist. You should add it in Commons in https://github.com/FossifyOrg/Commons/blob/master/commons/src/main/res/values/strings.xml. When creating a pull request in Commons, please mention that it's for this one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, I just have to add similar line there?? That's it? I created PR there too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's ok now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please provide compiled apk to test when you finish these changes.
@kumarayush2104 Hi! Do you know some Gujarati? If yes, can you test the suggested changes in this PR? |
@naveensingh I will test and provide you results as soon as possible |
@Abcd1234-dot @naveensingh Sorry for the delay. There are many problems in this pull request:
|
@kumarayush2104 Thank you for your testing. Here are my thoughts :
|
Sorry, closed it accidentally as a result of FossifyOrg/Commons#30 |
@Abcd1234-dot I resolved the conflicts and fixed some layout issues. I also removed the (unused) shift button from the Gujarati layout so you can fit in more characters on the bottom line if you want to. I don't speak the language myself so here's an APK: keyboard-fdroid-release.zip (GitHub doesn't support APKs, rename to keyboard.apk and install) |
@naveensingh Thank you!! I will revert you back within short time. Alright i tested the apk file you just tagged in previous reply. It is still very difficult and inconvenient to fit all characters properly within the space (even with no shift key!) (so either we have to use long press to pop-up which is not good for typing at all as @kumarayush2104 described or we can try the change i am proposing). So I propose a change in keyboard!! Rather than removing shift key, can we use it as "page 1/2" key?? Just like shift ON shows capital letters and shift OFF shows small letters (For English language), we can make shift key to show one set of (Gujarati) letters for ON state and another set of (Gujarati) letters for OFF state. This will solve similar "too many alphabets" issue for all languages permanently. I have added Gujarati Language to HeliBoard keyboard (another open source keyboard for android, fork of Openboard) with same functionality. I attach screenshots of Heliboard with Gujarati Language below (you can see how many characters to fit!!!). |
Yes but creating paged layouts will require some code restructuring. I'll be back to this after the initial release. |
@Abcd1234-dot Wouldn't it be better to follow the InScript standard with some modifications to make it mobile-friendly, instead of creating a different layout? |
It is not exactly "new layout", i made it very close to GBoard layout of same language with slight change, it would be convenient because most people would be switching from GBoard to other keyboards and GBoard layout makes typing easier on phone compared to desktop layouts of same language. I do not have technical knowledge to use or change InScript standard layout tho. |
What is it?
Description of the changes in your PR
Fixes the following issue(s)
Acknowledgement