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

added support for mini keyboard without m key #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ixeft
Copy link

@ixeft ixeft commented Jan 5, 2019

Hello,
I recently bought a mini keyboard
the problem is that to optimize space, they removed some key : it is impossible to type a «m» with this keyboard.

https://www.gearbest.com/other-consumer-electronics/pp_1443497.html?wid=1433363

In order to have a functional keyboard, and with the minimum code change in this app, I propose this pull request to add suport for mini keyboards (I assume that it is not the only model concerned)

suppressing «ç» to be able to type an «m»

@anisse
Copy link
Owner

anisse commented Jan 10, 2019

Thanks @ixeft , you can speak french as bépo is mostly a project for french typers; I'll assume you do.

J’aime bien l’idée de rajouter un autre clavier dans l’appli, c’est intéressant. J’ai par exemple un clavier (bluetooth) où le $ ou le # est impossible à faire, ça pourrait être une approche pour résoudre le problème.

Par contre le fichier .kcm est généré, en utilisant les outils officiels du projet bépo:
https://git.tuxfamily.org/dvorak/pilotes.git

L’idée c’est de mettre le script android.py dans le dossier configGenerator, et de l’executer avec en entrée le fichier xkb dans results/ généré par ./genAll.sh. J’aurais préféré que le script python soit directement intégré au projet, mais à l’époque je n’ai eu aucun retour des mainteneurs.

Pour la PR en elle même, l’idéal serait une fois celle-ci bien testée, de squasher les commits, sauf si il y a besoin de les séparer en plusieurs étapes.

@ixeft
Copy link
Author

ixeft commented Jan 11, 2019

d'accord, je vais regarder ça quand j'ai 5 minutes.

si il y a plusieurs configuration de clavier physique avec des touches manquantes, il faut voir si l'approche d'ajouter autant de disposition que de clavier physique est pertinente, j'imagine qu'à un moment ça risque de devenir un peu trop. Enfin pour l'instant on y est pas.

Par contre du coup, peut-être que le nom que j'ai donnée n'est pas idéal
«Français (bépo mini-clavier)»

faudrait une bonne convention de nommage 🤔

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