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

Word being replaced in emoji prediction #278

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

Word being replaced in emoji prediction #278

andrewtavis opened this issue Apr 2, 2023 · 8 comments
Assignees
Labels
-priority- High priority bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@andrewtavis
Copy link
Member

Terms

Behavior

Emoji autosuggestions were added via #51, but one lingering issue is that at times the word is being replaced rather than the emoji being added after it (similar to how an emoji autocompletion would replace the word, but this is not the desired behavior). An instance of this is when typing the word Kuss in German, meaning “kiss”, where when pushing either of the emoji suggestions the word is replaced by the emoji instead of the emoji being inserted after.

This is likely a conflict with the features implemented for #250 and #256 :)

Device type

iPhone and iPad

Versions

I’m assuming all versions

@andrewtavis andrewtavis added bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed -priority- High priority labels Apr 2, 2023
@andrewtavis andrewtavis self-assigned this Apr 2, 2023
@ArslanYM
Copy link

ArslanYM commented Apr 7, 2023

Is this issue fixed?

@andrewtavis
Copy link
Member Author

andrewtavis commented Apr 7, 2023

Hey @ArslanYM! Thanks for your interest in Scribe :) It’s yet to be fixed and I haven’t started working on it. Let me know if you have interest and I’d be happy to assign it to you 😊

@andrewtavis
Copy link
Member Author

@ArslanYM, I'm going to work on #277 today and think it makes sense if I work on this as well as I'll be testing this behavior :) There are a few more new issues that I made today that could also be of interest 😊

@andrewtavis
Copy link
Member Author

Obviously let me know if you have some code written though in the last few hours and you're welcome to commit it!

@andrewtavis
Copy link
Member Author

andrewtavis commented Apr 7, 2023

Note: specifically there's an issue where the word that would be displayed to a user in case the space autosuggest insertion was wrong is the word itself, but then it was the only option so the space should function normally in this case. allowUndo is possibly being triggered when it shouldn't, or currentPrefix is being incorrectly assigned and clearPrefixFromTextFieldProxy() is being triggered.

@ArslanYM
Copy link

ArslanYM commented Apr 7, 2023

No I haven't

@andrewtavis
Copy link
Member Author

Great :) Will send along a PR for this shortly 😊

@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-priority- High priority bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
Archived in project
Development

No branches or pull requests

2 participants