-
Notifications
You must be signed in to change notification settings - Fork 89
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
Celebration Fanfare shouldn't play for known appearances in Unique Mode #847
Comments
Lookalikes of class-specific items |
Ok, hmpf, my bad, looking back at it, I may have confused indicators from Can I Mog It? with ones from ATT, and/or phrased it the wrong way around altogether. Let's scratch that and reduce this to the Celebration sound. |
Currently this works as intended and as desired specified by the user. |
I don't particularly care about the jingle playing when you unlock a new source since technically a switch on the collection database is going from "I don't know this" to "I know this". Whether or not you know an appearance from a different source already at that moment would just add unnecessary lag or sometimes result in no jingle playing at all based on when the collection logic executed. Curiously though, what was your hotfix? |
For appearances: For pets: So, the only thing I could see causing additional significant CPU time would be the check for that new option, but I assume there's caching in place to minimize the CPU cost. Not that this code would be called often, from a computer's point of view ;) |
I agree with the points of this issue, and both Appearances (in Unique Modes) and Battle Pets properly only play their fanfare based on whether you have collected something new. |
"Collected Things Trigger a Celebration" in Unique Mode
The fanfare is great, but in Unique Mode, it shouldn't trigger for sources where you already collected a shared Appearance. Could be controlled by a secondary option.
It can be misleading, especially when you get a random item, e.g. from a token or chest.
PS: Similarly, learning copies of already collected Battle Pets also trigger the fanfare. I would consider 1/3 being sufficiently collected for my purposes.
Lookalikes of class-specific itemsIf you have collected a class-specific appearance, e.g. a Tier item, but not any lookalikes, it shows these lookalikes as "collected from another appearance" on other classes, but you couldn't use this look on other classes. Similar for the PvP sets in WoD, where each faction got the other faction's looks the following season.
Could these cases be marked differently? I could imagine a yellow tint of the "known_circle" icon.
The text was updated successfully, but these errors were encountered: