Change Matrix3d transform into Translate & Scale transforms

HeartofPhos / exile-leveling

Path of Exile Leveling Guide

https://HeartofPhos.github.io/exile-leveling/

159 stars 31 forks source link

Change Matrix3d transform into Translate & Scale transforms #63

Closed ennukee closed 1 year ago

ennukee commented 1 year ago

This should help reduce the browser load when calculating the repositioning of the tree viewer on devices with lower specs (e.g. mobile devices).

HeartofPhos commented 1 year ago

Testing on Desktop (Firefox, Edge) and Mobile (Firefox Android) I'm not seeing an meaningful difference in performance between the 2 options, if there is noticeable difference on Safari I'm happy to change it

ennukee commented 1 year ago

The performance increase looked to be primarily on mobile devices. My 'older' iOS devices were struggling to render the large matrix3d transform and were crashing as a result.