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

WhoReacted should not include hidden reactions in its counts #403

Open
FichteFoll opened this issue Apr 2, 2023 · 2 comments
Open

WhoReacted should not include hidden reactions in its counts #403

FichteFoll opened this issue Apr 2, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@FichteFoll
Copy link
Contributor

In the current version of WhoReacted, when you activate the "Hide self" and "Hide bots" options, the +N counts will still include them. This is inconsistent because this doesn't happen when there are only ignored reactions on a message.

Only ignored reactions: 2023-04-02_20-39-45

One ignored and two other reactions: 2023-04-02_20-40-09

Furthermore, I am never interested in this ignored reactions increasing the "+N" number because I specifically want to hide them entirely (which is why I implemented it this way in the first place in #4) and if I really wanted to know how many reactions there are in total, I could just look at the number on the left which will always be there.

In my option, it makes no sense to include ignored reactions in the +N counts in any situation.

@jaimeadf jaimeadf added the enhancement New feature or request label Apr 3, 2023
@jaimeadf
Copy link
Owner

jaimeadf commented Apr 3, 2023

I must agree with you after using WhoReacted for the last two days. The "+N" is useless and just occupies precious space most of the time. However, I think that when the "+N" appears, the hidden reactions should be counted as it is impossible to know if the people who haven't been fetched meet the filter's criteria. I'm going to remove the "+N" when the maximum avatar limit hasn't been reached and maybe add an option to fully disable it. In the latest version, I've moved away from using discord components and have full control over everything that is rendered, which means there are no limits on what can be modified.

@FichteFoll
Copy link
Contributor Author

I think that when the "+N" appears, the hidden reactions should be counted as it is impossible to know if the people who haven't been fetched meet the filter's criteria

I presume this is because you only fetch as many user details as you need to reach the number of maximum avatars to be displayed? Could I interest you into adding an option to perform such fetches for as many as the configured maximum for WhoReacted to operate for a single reaction? In my situation, I have very rarely more than 12 users on a single reaction and most of the times one of them is going to be a bot that I'm very not interesed in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants