azerothcore / mod-transmog

Plug&Play transmog module for AzerothCore, based on Rochet2 works
https://github.com/azerothcore/mod-transmog/blob/master/README.md
GNU Affero General Public License v3.0
120 stars 144 forks source link

Suggestion: Transmogs for cross-armor type #142

Open Aldori15 opened 8 months ago

Aldori15 commented 8 months ago

Would it be possible to implement the ability to learn transmogs for different armor types? For example, playing on a Plate character can learn transmogs for Cloth or Leather when they are looted. And then logging over to a Cloth or Leather character later will already have those appearances unlocked? That way you don't have to run a dungeon on 4 different characters of 4 different armor types to get all of the appearances.

Or playing on a Cloth character can still unlock the appearances of Plate armor in the collection upon looting the item. Point is, no matter what armor type the player is, if they could be able to unlock appearances for any armor type.

Instead of only Plate-->Plate, Mail-->Mail, Leather-->Leather, Cloth-->Cloth.

Maybe this could be done via a setting in the config for people to turn on/off if they didn't want it?

Helias commented 8 months ago

there is already a conf about this

Transmogrification.AllowMixedArmorTypes = 0

does make what you are looking for?

About sharing the appearance between characters of the same account I think it's another feature request that deserves a different issue

Aldori15 commented 8 months ago

Yes, I have Transmogrification.AllowMixedArmorTypes = 1, but it's a bit different than what I'm looking for. That config option allows the ability to transmog different armor types after they are already learned.

But what i'm looking for is the ability to learn appearances no matter what armor type. If i'm playing on a Plate character and a piece of Cloth armor drops, I can't learn that appearance unless i'm on a Cloth character.

Ceikry commented 3 months ago

Possibly resolved by #158

Helias commented 3 months ago

@Aldori15 if you could verify that the issue does not persist anymore with the new configuration please let us know and if so, close the issue

Aldori15 commented 3 months ago

@Helias Will do. I'll pull down the latest and let you know.

Aldori15 commented 3 months ago

@Helias The new "Transmogrification.AllowLowerTiers" config option is exactly what we were looking for. Playing on a Warrior (plate class) and a Cloth piece drops. Once I equip the Cloth piece, it learns that appearance. Great, thanks!

I'm not sure if these are even possible or not given you currently have to equip the piece of armor in order to learn the appearance. And since you can't equip higher tier armor pieces, I'm not sure if it's in the realm of possibility, but figured I would ask anyway since you guys know more than I do.

Ceikry commented 3 months ago

I don't believe it's possible, no. Some method of unlocking appearances in a way other than equipping the gear would have to be added. I (think) there's a GM command or something? But that's probably not in the realm of practicality for your players

Ceikry commented 3 months ago

I can potentially visit the idea of an auto-unlock-on-loot toggle in a future MR, so that you could do it that way. Fair warning though, you would unlock a TON of appearances that way that you probably wouldn't care about.

Aldori15 commented 3 months ago

Yeah, gm command wouldn't work. If it's not possible, then I wouldn't sweat it much. The setting you guys have implemented fixes about half the problem. If you want to visit the idea of an auto-unlock-on-loot toggle, that would be awesome.

I'm not too worried about unlocking a bunch of appearances. The whole idea really is being able to run dungeon/raid content on any class armor type and if loot drops that isn't your armor type, you can still learn those transmog appearances for other characters. That way you don't have to run the same dungeon on 4 different armor types to get all of the appearances.