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

Allow for swiping between conjugation and declension views #361

Closed
2 tasks done
andrewtavis opened this issue Sep 18, 2023 · 10 comments
Closed
2 tasks done

Allow for swiping between conjugation and declension views #361

andrewtavis opened this issue Sep 18, 2023 · 10 comments
Assignees
Labels
feature New feature or request help wanted Extra attention is needed

Comments

@andrewtavis
Copy link
Member

Terms

Description

This issue would add swipe functionality to the conjugation and declension views for Scribe keyboards, with the former being displayed when a user presses the conjugate key and enters a verb and the latter being displayed when a user clicks on a preposition's case to see the possible pronouns that could be used. As of now we have a left and right button to the side of the view that are used to switch views, with the goal being that the user would also be able to swipe. The left and right buttons would not be removed to make sure that the user can still understand the ability to switch views :)

The code for this would generally follow the patterns used in #337 that added swipe support to the new menu 🚀

Contribution

Happy to work on this with someone or get to it myself! 😊

@andrewtavis andrewtavis added feature New feature or request help wanted Extra attention is needed labels Sep 18, 2023
@andrewtavis
Copy link
Member Author

CC @SaurabhJamadagni on this :)

@henrikth93 henrikth93 self-assigned this Apr 6, 2024
@andrewtavis
Copy link
Member Author

CC @Taufi, can you write in here so I can assign you as well :)

@Taufi
Copy link
Contributor

Taufi commented Apr 8, 2024

I would like to work with Henrik on the issue. Is this the information you need?

@andrewtavis
Copy link
Member Author

Exactly, @Taufi :) As soon as you write in I can assign you. Helps us keep an overview of who's on what 😊

Let me know if there's anything I can do to help!

@Taufi
Copy link
Contributor

Taufi commented Apr 8, 2024

Yes, you can help us:
Henrik and I want to add a swipe gesture recognizer to the keyboardView. We are uncertain about the optimal location for defining it. ViewDidLoad in KeyboardViewContoller seems appropriate since it's only called once. Although setConjugationBtns is closer in context to conjugation, it's called frequently. Do you have any suggestions?

@andrewtavis
Copy link
Member Author

Thanks for the research so far, both of you! 🙌 Both are really good points!

Originally I was typing out that ViewDidLoad makes sense to keep the keyboard load times down, but that's also with the assumption that a person will use conjugate at least once when using Scribe. Maybe for Scribe-Desktop where someone's writing an email or a paper, but I'd say that in a typical phone interaction a person is using verbs that they generally know how to conjugate, and just need help in special circumstances. In this case I'd go with setConjugationBtns, with this also having the architectural bonus that its usage is closer to its function :)

How does this sound?

@Taufi
Copy link
Contributor

Taufi commented Apr 9, 2024

Your reasoning convinces me, Andrew. I think that's how we should proceed. Many thanks for the tip.

@andrewtavis
Copy link
Member Author

Many thanks for checking in, @Taufi!

@henrikth93
Copy link
Collaborator

Thanks for the research so far, both of you! 🙌 Both are really good points!

Originally I was typing out that ViewDidLoad makes sense to keep the keyboard load times down, but that's also with the assumption that a person will use conjugate at least once when using Scribe. Maybe for Scribe-Desktop where someone's writing an email or a paper, but I'd say that in a typical phone interaction a person is using verbs that they generally know how to conjugate, and just need help in special circumstances. In this case I'd go with setConjugationBtns, with this also having the architectural bonus that its usage is closer to its function :)

How does this sound?

Makes sense!

Taufi added a commit to Taufi/Scribe-iOS that referenced this issue Apr 12, 2024
andrewtavis added a commit to Taufi/Scribe-iOS that referenced this issue Apr 13, 2024
andrewtavis added a commit that referenced this issue Apr 13, 2024
Allow for swiping between conjugation and declension views #361
@andrewtavis
Copy link
Member Author

Closed by #412 🥳 Thanks so much, @Taufi and @henrikth93 for the great collaboration on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request help wanted Extra attention is needed
Projects
Archived in project
Development

No branches or pull requests

3 participants