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

Fix noun article on annotation press #302

Open
2 tasks done
andrewtavis opened this issue May 12, 2023 · 8 comments
Open
2 tasks done

Fix noun article on annotation press #302

andrewtavis opened this issue May 12, 2023 · 8 comments
Assignees
Labels
-priority- High priority feature New feature or request help wanted Extra attention is needed

Comments

@andrewtavis
Copy link
Member

Terms

Description

Amazing feature idea that came up via showing Scribe to a coworker 🎉 It would be great if we added a feature where pressing the noun annotation would check to see if there’s an article before it and if so if it’s the incorrect article. If it’s not correct then the word and the article can then be replaced with the correct one and the word after it 😊

Things to consider:

  • We’d need to make sure the key gets the shadow, which I believe comes from the active state regardless
  • For words that are both nouns and prepositions the article correction would first be triggered and then if the article is correct we could do preposition pronoun selections
  • It’d be good if the replacement was matched with an indicator to the user, with the current working thought being that we could display a checkmark icon on the annotation for a moment before reverting back to the annotation

This feature would definitely be worth a release for v2.4 🚀

Contribution

I’ll start working on this soon and would be happy to get some support :)

@andrewtavis andrewtavis added feature New feature or request help wanted Extra attention is needed -next release- Included in the next release labels May 12, 2023
@andrewtavis andrewtavis self-assigned this May 12, 2023
@andrewtavis
Copy link
Member Author

@SaurabhJamadagni, @wkyoshida: tagging to bring this to your two’s attention as this would be really great :) One of those can’t believe we didn’t think of this features 😊 Consistently during demos I’m telling people that they’d at least know that they need to go back and change the article if it’s wrong, so this would be huge!

@andrewtavis
Copy link
Member Author

An extra level of complexity for this comes when we consider definite and indefinite articles, so this should also be checked for. Beyond that is also the question of if there’s a certain case that is being used in German and Russian. We haven’t done verb-case annotation in #81, but for prepositions we could also check to see if there’s one in the sentence and then select the appropriate article. If the preposition only has one case then we can insert it, and if there is more than one possible then we can do a selection journey potentially :)

@andrewtavis
Copy link
Member Author

andrewtavis commented May 12, 2023

Something to consider for this is that we also need to factor for words that could have more than one gender — for instance the classic example for Scribe that Schild in German is masculine when it means shield and neutral when it means sign — such that the user would first be prompted to select the correct gender. Beyond this if there is more than one case possible then we don't necessarily want to show the options for other genders in the normal selection as that would allow them to insert an incorrect value. In cases where there is more than one gender possible and it's after a preposition that has more than one case, then they should first select the gender and then the case.

@andrewtavis
Copy link
Member Author

For words that have more than one gender we should set it up so that clicking the annotation brings up a display to check/click the right one. This would then allow us to later add functionality where the user could check if they've used the right article given their meaning via the importing of the senses of words such that we could also display the sense as a definition to the user.

@SaurabhJamadagni
Copy link
Collaborator

Hey @andrewtavis, sounds like a great feature idea! I am kinda lost on the intricacies though. Could you explain them to me when we have our call? 😅

@andrewtavis
Copy link
Member Author

Yes yes, definitely can explain more, @SaurabhJamadagni 😊 As you can see was one of those ideas that kind of snowballed after we originally came up with it 😅😊😊

I’ll plan on getting the emulator out for the call to explain what would happen :)

@andrewtavis
Copy link
Member Author

Based on a comment from @SaurabhJamadagni, the checkmark should be displayed even if the article is correct. We decided with @wkyoshida to do the checkmark if it has been corrected, and a thumbs up if it was correct in the first place. These symbols would come from SFSymbols :)

@andrewtavis andrewtavis added -priority- High priority and removed -next release- Included in the next release labels Sep 18, 2023
@andrewtavis
Copy link
Member Author

Something to potentially add to this would be a quick animation within the command area that shows the correct article and the noun together and colored with the given gender color. An example:

  • The user types "der Buch" on a German Scribe keyboard (should be "das")
  • The users sees the N annotation and realizes that "der" is wrong
  • The user presses the Scribe key and the article is replaced
  • As confirmation of the command, the command bar is shown for a second with "das Buch" as the text on it
  • The command bar's background color is green for neutral nouns, and the text is the same color as the annotation text color

I think that this would be a good way of doing this such that we're reinforcing the correct gender with the quick fix command :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-priority- High priority feature New feature or request help wanted Extra attention is needed
Projects
Status: Todo
Development

No branches or pull requests

2 participants