snowl / LegendaryClient

Custom League of Legends client. No longer in development.
MIT License
43 stars 30 forks source link

Summoner Icon 29 missing in Assets/profileicon #13

Closed jonnyli1125 closed 10 years ago

jonnyli1125 commented 10 years ago

It's not in the unzipped archive either.

snowl commented 10 years ago

What icon is 29?

jonnyli1125 commented 10 years ago

summoner icon 29

snowl commented 10 years ago

Are you sure thats not icon 0?

jonnyli1125 commented 10 years ago

The client crashed because it was trying to load 29, but 29 didn't exist? Guess I'll just copy 0 to 29 derp

snowl commented 10 years ago

when did it crash?

snowl commented 10 years ago

I can't replicate it

example

SaintDroid commented 10 years ago

I've got the same problem but with the icon 37. I checked on the original LOL client and I saw that the missing 37.png file is the 0.png image. Maybe that icon has more names (actually 0, 29, 37).

snowl commented 10 years ago

I stopped it crashing so I'll close it but I don't know what the underlying problem is...

jonnyli1125 commented 10 years ago

How about just make any icon that doesn't exist use icon 0 (just in case it shows up again or something)?