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

Case annotations after contracted prepositions #279

Closed
2 tasks done
andrewtavis opened this issue Apr 5, 2023 · 10 comments
Closed
2 tasks done

Case annotations after contracted prepositions #279

andrewtavis opened this issue Apr 5, 2023 · 10 comments
Assignees
Labels
feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@andrewtavis
Copy link
Member

Terms

Description

Scribe shows case annotations after German prepositions like “in”, “für” and “zu”. As of now it does now show them on cases where a combined preposition has been typed like “zu” + “der” = “zur”. This issue would add a condition to show the appropriate preposition annotations after such words. This issue could also be done in Python in Scribe-Data as well in a way that would simply generate a JSON file with the appropriate data to also include these :)

Contribution

Happy to work on this or support someone who has interest :)

@andrewtavis andrewtavis added feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Apr 5, 2023
@andrewtavis
Copy link
Member Author

The person has potentially chosen the right option and doesn't need the annotation, but the behavior of showing the annotations after prepositions should be maintained.

@ArslanYM
Copy link

ArslanYM commented Apr 7, 2023

@andrewtavis Can i work on this?

@andrewtavis
Copy link
Member Author

Hey @ArslanYM! Thanks for your interest both here and in #278 😊 You’d be welcome to work on this. Let’s chat quickly on what we think is the best way to do this. Should we add entries to the prepositions dictionary on the Swift side, or should this be handled on data preparation? I’m fine with both, and I guess doing the former would be easier and shouldn’t be frowned upon as the data output is sound: “zur” in itself isn’t a preposition.

If this sounds good, then I think the change would go into loadKeys in the main view controller — first checking if the keyboard is German, and then in the initial keyboard load step we’d add the entries to prepositions :)

Lemme know what you think! Assigning you now 😊

@andrewtavis
Copy link
Member Author

This line right here is the conditional that checks if it's the first time the keyboard is loading. Within there is generally where I put the logic that extends the dictionaries and such. You can see in the line below that's in there how we'd add entries:

nouns["Scribe"] = [
        "plural": "Scribes",
        "form": ""
      ]

We'd then generally just need to define which preposition combinations need to be included, check for the German keyboard, and add them in the above way :)

@ArslanYM
Copy link

ArslanYM commented Apr 7, 2023

@andrewtavis Sorry for bothering. I don't think i can work on this.

@andrewtavis
Copy link
Member Author

No bother! Happy to help you figure it out of you have interest, but no stress if not :)

@andrewtavis andrewtavis changed the title Case annotations after combined prepositions Case annotations after contracted prepositions Apr 7, 2023
@andrewtavis
Copy link
Member Author

Closed in 97c0afb 🚀

@github-project-automation github-project-automation bot moved this from Todo to Done in Scribe Board Apr 7, 2023
@andrewtavis andrewtavis assigned andrewtavis and unassigned ArslanYM Apr 7, 2023
@andrewtavis
Copy link
Member Author

Thanks again for chatting with me about this, @ArslanYM! Was really helpful to explain it and plan it out for myself 😊 If you're looking for a good first issue, then #280 would be a great one to jump in to :)

Have a nice weekend!

@ArslanYM
Copy link

ArslanYM commented Apr 7, 2023

Sure thing. Thankyou for being a great mentor.

@andrewtavis
Copy link
Member Author

My pleasure! Thanks again for your interest 😊 Let me know in #280 if you want to be assigned, and again very happy to explain or discuss further :D

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

No branches or pull requests

2 participants