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

Boss Tracker: consider removing prioritizePremium #150

Open
mathiasbynens opened this issue Dec 13, 2022 · 0 comments
Open

Boss Tracker: consider removing prioritizePremium #150

mathiasbynens opened this issue Dec 13, 2022 · 0 comments

Comments

@mathiasbynens
Copy link
Contributor

mathiasbynens commented Dec 13, 2022

() => prioritizePremium(listBy[listingOption](hydratedBossList)),

Why are premium bosses prioritized? As an Exevo Pro user, I would prefer all bosses to show up based on their % chance or whatever listingOption was chosen (i.e. the way it worked before this change). I understand you want to highlight premium bosses in a special way, but the styling already accomplishes that IMHO. It's not helpful to push the premium bosses to the start if the list even if there's no chance of them spawning on my server today.

Here's what I'm seeing for my list of Favorites for my server:

None of the premium bosses have any chance of spawning today, yet they take up space at the start of the list.

Would you be willing to reconsider this? The code change would be simple -- just don't use prioritizePremium at all.

@mathiasbynens mathiasbynens changed the title Consider removing prioritizePremium Boss Tracker: Consider removing prioritizePremium Dec 13, 2022
@mathiasbynens mathiasbynens changed the title Boss Tracker: Consider removing prioritizePremium Boss Tracker: consider removing prioritizePremium Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant