Skip to content
This repository has been archived by the owner on Nov 6, 2019. It is now read-only.

Max size for champion portrait and scaling fix #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

corylulu
Copy link
Contributor

Currently, Wukong's portrait is loading the wrong image from ddragon. As seen here: http://champion.gg/champion/MonkeyKing (as of 6.1.1 patch).

It's just loading a 1000x1000px green box and because there is no size limit on the site, it's covering the whole page. This change will prevent that from happening again.

Also for the small champion portraits, added "translateZ(0)" to the transform property to fix the blurriness on Chrome.

Currently, Wukong's portrait is loading the wrong image from ddragon. As seen here: http://champion.gg/champion/MonkeyKing (as of 6.1.1 patch). 

It's just loading a 1000x1000px green box and because there is no size limit on the site, it's covering the whole page. This change will prevent that from happening again. 

Also for the small champion portraits, added "translateZ(0)" to the transform property to fix the blurriness on Chrome.
Using "translateZ(0)" or changing the scaling from 0.7 => 0.75 both fix the blurriness issue in Chrome, but might as well do both. Last time I fixed it with 0.75 scaling, but missed that there were two classes I needed to change.
@corylulu corylulu mentioned this pull request Jan 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant