PhilippC / kp2a_keyboardswap

Keepass2Android KeyboardSwap Plugin
GNU General Public License v3.0
27 stars 1 forks source link

Avoid missunderstanding how to use ADB Command (design improvement) #4

Open flohall opened 6 years ago

flohall commented 6 years ago

First A big thank you for your great work. I want to suggest a little improvement for the keyboardswap2 plugin.

Problem in short: For me (I am not an android developer) it was not obvious that I have to put a SPACE between keyboardswap2 and android in the pm grant command for adb shell. I found out this after checking how the manifest file (out of source code) is structured, but maybe other users will give up faster.

More details about the problem: In the keyboardswap2 plugin instructions under no 4 the command looks like it is quite long and that there is just a linewrap. It is not easy to recognize that there should be a SPACE at the end of the line. But the correct command is: pm grant keepass2android.plugin.keyboardswap2 android.permission.WRITE_SECURE_SETTINGS Maybe this issue is only crucial on some devices with special screen width. I use Huawei P10.

Hard and Software used Huawei P10 VTR-L09 EMUI-Version: 8.0.0 Android-Version: 8.0.0 Screen: 1920*1080

Possible solutions: