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

tabcomplete: emotes vs usernames #62

Open
xDashh opened this issue Dec 8, 2018 · 5 comments
Open

tabcomplete: emotes vs usernames #62

xDashh opened this issue Dec 8, 2018 · 5 comments
Labels
bug Something isn't working

Comments

@xDashh
Copy link
Member

xDashh commented Dec 8, 2018

emote(s) always take preference (even if not used lately) before users (even if recently tabbed) currently. Expected: tabbed users should be before un-tabbed emotes.
Making tabbed users a preference before tabbed emotes might also be worth considering since you probably care more about the user once you talk, but it's an inconsistency vs. the un-tabbed case...

In the new dev-chat this can be seen by tabbing "pepo" (there are various emotes, and 2 users by that prefix).

@xDashh xDashh added the bug Something isn't working label Feb 10, 2019
@SmCTwelve
Copy link
Contributor

I can fix this to satisfy the following conditions:

  • untabbed user vs untabbed emote -> user first
  • tabbed user vs untabbed emote -> user first
  • untabbed user vs tabbed emote -> emote first
  • tabbed user vs tabbed emote -> most recently used first

Is that what you're looking for? The sort function already contains the logic for this but it just places emotes first regardless of usage because of an early return.

I'm wondering what CUSTOM_AUTOCOMPLETE_ORDER is used for? Is it a dev thing? Or is there a command people can use to control it? Currently it overrides any other sorting which seems to be expected behaviour. ['smctwelve', 'SMOrc'] for example 😏.

@xDashh
Copy link
Member Author

xDashh commented Jul 31, 2019

The CUSTOM_AUTOCOMPLETE_ORDER is used to preserved order for things that many people are used to, e.g. REE existed for years, while RedCard is pretty new, so people were used to re+tab completing to REE first. After some time some custom orders are removed as they become less necessary.

Otherwise I'm not sure atm, but this should be good https://github.com/MemeLabs/Rustla2/wiki/Chat-Autocomplete

@xDashh
Copy link
Member Author

xDashh commented Jul 31, 2019

While you are looking at this code, maybe #109 could be done too?

@SmCTwelve
Copy link
Contributor

SmCTwelve commented Jul 31, 2019

I have a fix for that already I'll post it soon 👍. I tried looking into #19 too but I can't track that behaviour down.

The custom order thing makes sense. So with that wiki description I think the 2nd, 3rd and 4th conditions posted above satisfy. So in the case of both a user and emote being untabbed, is it expected for the emote to come first rather than a user?

@xDashh
Copy link
Member Author

xDashh commented Jul 31, 2019

I think so, yeah. Because you don't know an unknown user, but you probably know a lot of emotes you haven't tabbed recently. There might be something about giving a user that wrote in chat recently preference over an unused emote but that might've caused trouble, don't remember. The Wiki should have everything I knew when I was thinking about this a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants