Recipe Master

Recipe Master

9.3k Downloads

Characters are not presented in alphabetical order in recipes' tooltip

Closed this issue · 4 comments

commented

Thanks for doing this!

Pasted_Image_10_25_24__2_08 PM

It seems to be capped off at listing 5 toons. It'd be better if it listed all that it knew about on the account, I think. Maybe give people an option to limit to 5, but I'd love to see all the toons on there. (=

commented

Hmm, actually I think it's missing the current toon from the list. I'm on the Druid here, but it's not showing as missing for the Druid.

image

And it'd be good if the list could be alphabetized. (=

commented

Hello, @Gogo1951.

I'm glad you've been enjoying the feature :)

I haven't implemented a limit to how many characters are displayed, but a character must have the profession learned in order to be in one of the lists. Could that be the case?

About the current character not being on any of the lists: this was a deliberate choice to avoid redundancy since "Already known" is displayed in red on the tooltip.

commented

Yeah I figured it was a design choice. I added a simple alphabetical sort in a PR.

I just came back to Classic Era about 2 months ago... just leveling new toons and enjoying myself. Messing around mostly. This feature is great for helping me collect recipes. Thanks!

commented

My pleasure! And thank you for the pull request!

The feature is now implemented in version 2.4.1.