Skip to content

Fix(general-keyboard): Automatically switch to letter keys after command selection#299

Merged
andrewtavis merged 2 commits intoscribe-org:mainfrom
smalik2811:fix-keyboard-switch-after-command-selection
Jan 19, 2025
Merged

Fix(general-keyboard): Automatically switch to letter keys after command selection#299
andrewtavis merged 2 commits intoscribe-org:mainfrom
smalik2811:fix-keyboard-switch-after-command-selection

Conversation

@smalik2811
Copy link
Contributor

Contributor checklist


Description

This commit addresses issue #298 by:

  1. Automatically switching the keyboard to 'keyboardLetters' after a Scribe command (e.g., translate, conjugate, plural) is selected. This improves the user experience by streamlining the workflow and reducing the need for manual keyboard mode changes.

  2. Switching the keyboard to 'keyboardLetters' when the user switches to the Idle mode. This ensures consistent keyboard behavior and prevents unexpected input modes.

This enhancement enhances the overall usability of Scribe by providing a more intuitive and efficient interaction flow for users.

Closes #298

Related issue

This commit addresses issue scribe-org#298 by:

1. **Automatically switching the keyboard to 'keyboardLetters' after a Scribe command (e.g., translate, conjugate, plural) is selected.** This improves the user experience by streamlining the workflow and reducing the need for manual keyboard mode changes.

2. **Switching the keyboard to 'keyboardLetters' when the user switches to the Idle mode.** This ensures consistent keyboard behavior and prevents unexpected input modes.

This enhancement enhances the overall usability of Scribe by providing a more intuitive and efficient interaction flow for users.

Closes scribe-org#298
@github-actions
Copy link

github-actions bot commented Jan 17, 2025

Thank you for the pull request!

The Scribe team will do our best to address your contribution as soon as we can. The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :)

If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the General and Android rooms once you're in. Also consider joining our bi-weekly Saturday dev syncs. It'd be great to have you!

Maintainer checklist

  • The linting and formatting workflows within the PR checks do not indicate new errors in the files changed

  • The CHANGELOG has been updated with a description of the changes for the upcoming release and the corresponding issue (if necessary)

@andrewtavis andrewtavis self-requested a review January 17, 2025 10:27
@andrewtavis
Copy link
Member

Minor feedback here, @smalik2811: I think that we don't need the keyboard switching back to the letter keys each time the state goes to idle. Wasn't quite as clear on this as I could have been :)

An example would be:

  • The user is on symbols
  • The user presses the Scribe key
  • The user presses exit as they're exploring the keyboard for the first time or just decided not to use a command
  • In this case the keyboard should still be symbols

What I meant was that each time after the user executes a command that we should switch back to the letter keys, but then that would be handled in the functions that execute the commands, not on a switch to letters whenever the keyboard switches to idle mode.

Do you want to send along the change such that this just switches the keyboard to letters when we enter a command mode, and then we can keep the letter keys switch in mind for when we work on the command functions in #266, #270 and #272?

Thanks for the help with this! 😊

CC @angrezichatterbox on this as well 👋

@angrezichatterbox
Copy link
Member

So does this mean that the keyboard should be on Keys on all modes except select command mode and idle mode. Like when the user click on a particular command translate plural or conjugate he is switched to the keys. Am I right ?

@andrewtavis
Copy link
Member

Yes, exactly, @angrezichatterbox :) If they're on symbols and want to enter a word into a command, then the first thing they'll need to do is switch to letters, and they'll need to do it basically always, so let's do it for them 😊

This commit addresses issue scribe-org#298 by:

1. **Automatically switching the keyboard to 'keyboardLetters' when the user enters the command mode.** This improves the user experience by streamlining the workflow and reducing the need for manual keyboard mode changes.

This enhancement enhances the overall usability of Scribe by providing a more intuitive and efficient interaction flow for users.

Closes scribe-org#298
@smalik2811
Copy link
Contributor Author

Hey @andrewtavis, thanks for the feedback 😅. I totally missed the point that the user can return back to the Idle mode straight from the Command Select mode and this could result in the issue of unnecessaryly switching back from symbols to letters.

I have removed the part causing the issue. Now changes only includes the code to switch symbols to letters then the user actually enters the command mode.

The changes were affecting the capitalization of Noun in German, I have also fixed that.

@andrewtavis
Copy link
Member

So far so good, @smalik2811! Thanks for the quick reaction :) One final comment here is that the German noun capitalization is still being effected by this, or maybe it's broken and we need to fix it. When we do the plural command on the German keyboard the keyboard should switch to letter keys and it should also be capitalized for one key.

Do you want to try to send along a fix for this? 😊

@smalik2811
Copy link
Contributor Author

Hey @andrewtavis, I tested the app on a virtual device. I first switched to the symbol keyboard from the idle mode and then entered the Plural mode. The keyboard switched to the letters keyboard and the letters are capitalized as expected.

German.capitalization.webm

I must have missed something 😅, could you specify the issue you noticed with the German noun capitalization.

Copy link
Member

@andrewtavis andrewtavis left a comment

Choose a reason for hiding this comment

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

Sorry for the confusion, @smalik2811! Thanks much for the video showing that all's working. I just updated to Android Studio Ladybug and I think that there was just some issues where the changes weren't being reflected. Needed to leave the keyboard and come back to it and it's all great now 😊

Thanks for the continued support!

@andrewtavis andrewtavis merged commit c7898dc into scribe-org:main Jan 19, 2025
4 checks passed
@smalik2811
Copy link
Contributor Author

No problem at all! Glad you got it sorted out and we have resolved another issue 😊

@smalik2811 smalik2811 deleted the fix-keyboard-switch-after-command-selection branch January 25, 2025 02:24
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.

Switch keyboard to letters when a command mode command is selected

3 participants