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

Highlight autocompletion if it is the word typed #250

Closed
2 tasks done
andrewtavis opened this issue Nov 6, 2022 · 14 comments
Closed
2 tasks done

Highlight autocompletion if it is the word typed #250

andrewtavis opened this issue Nov 6, 2022 · 14 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

This issue a a simple one to reflect system behavior where when the keyboard suggests the exact word the user has typed as a completion it's usually enclosed in quotes. What Scribe could do for this is lightly change the background of the autocompletion if the word is the given word. This is also done in some other keyboard apps, and I think it's more appealing than the word in quotes.

The background of the completion key would thus need to be conditionally changed and reset as the user continues typing :)

Contribution

Happy to support someone who wants to try this issue 😊

@andrewtavis andrewtavis added feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Nov 6, 2022
@techsandu
Copy link

Hey @andrewtavis can you please assign this in my name? I will look into this and come back.

@andrewtavis
Copy link
Member Author

Sure thing, @techsandu! Thanks for reaching out!

Would you like me to give you a detailed breakdown of where to look in the codes? Happy to send something along to contextualize this for you 😊

@andrewtavis
Copy link
Member Author

In case it would be helpful, the assignment of the word is within this function:

func conditionallySetAutoActionBtns() {

So basically we need to do a comparison of the word that's in the proxy with completionWords[0], as naturally the first suggestion would be the one we want a slightly different coloration for given alphabetization. If memory serves me, currentPrefix also saves the value being typed in any given moment, so that would be what we're comparing completionWords[0] to, and in case they are the same we'd be changing the color argument of the function setBtn in conditionallySetAutoActionBtns linked to above 😊

As far as a color, I'm open to suggestions, but maybe something like keyColor with an alpha would be in order? Would be very happy to answer any other questions you have about this!

@andrewtavis
Copy link
Member Author

@techsandu, checking in with you on this :) Let me know if you need any assistance 😊

@cameronslee
Copy link

Hi @andrewtavis, Could you assign this issue to me? This is my first contribution to an open source project and I have a pull request ready to submit. I'm still a little bit confused about the process of contributing so any guidance would be much appreciated!

@andrewtavis
Copy link
Member Author

andrewtavis commented Nov 27, 2022

Happy to assign it given you have a PR, @cameronslee23 :) I'll check the contribution later on today and get back to you, but so far you're following the process very well! 😊

@andrewtavis andrewtavis assigned cameronslee and unassigned techsandu Nov 27, 2022
@andrewtavis
Copy link
Member Author

PR was exactly what we're looking for in contributions, @cameronslee23 😊 Thanks for updating the changelog as well. The only thing I added was a period at the end of the entry in there 🙃 Hope to work with you more in the future! 🚀

@andrewtavis
Copy link
Member Author

@cameronslee23, one thing to check is whether your email is set up correctly for GitHub. You're currently not listed in the contributors. This Stack Overflow question talks about it :) Would be great if you could fix that as you'd be our 10th contributor, which would actually allow me to apply to get Scribe onto a list of projects that are good for new developers (10 contributor minimum) 😊

@cameronslee
Copy link

@andrewtavis Thank you! I updated my email settings so hopefully I show up on the list soon! Hope to work with you in the future as well :)

@andrewtavis
Copy link
Member Author

You should be in there in a few days, @cameronslee23 :) Sometimes takes a bit for this setting to update 😊

Thanks again!

@andrewtavis
Copy link
Member Author

andrewtavis commented Dec 2, 2022

@cameronslee23, you're still not in the contributors, which is unexpected at this point 🤔 I do want you to get your due credit 😊

Besides this, just wanted to let you know that this feature unlocked another one in #256! 🥳 Might be an easy one for you to do if you had interest, but aside from that thanks again for the work here that allows us to add other cool things! 🚀

@cameronslee
Copy link

@andrewtavis Yeah I'm not sure why I'm not on the list, all of the settings check out correctly to my understanding. I just took a look at #256 and would be happy to work on it!

@andrewtavis
Copy link
Member Author

Maybe it’s just gonna take a bit :) We’ll see 😊 Will check in with you on #256 over there! 🚀

@andrewtavis
Copy link
Member Author

If the email that you use for GitHub is the same that you have for git config --global user.email "[email protected]" then we should be good :) Might be that the first commit can’t be applied to your account, so you’ll be in there after #256 :)

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
None yet
Development

No branches or pull requests

3 participants