search

monome / maiden

web based editor and repl for norns
GNU General Public License v3.0
47 stars 33 forks source link

Fix `scrollTo` error in MS Edge (fixes #157) #165

Closed synthetiv closed 4 years ago

synthetiv commented 4 years ago

Picking more low-hanging fruit; apparently MS didn't bother to implement element.scrollTo(), but setting scrollTop is a viable alternative.

synthetiv commented 4 years ago

Oh, I should have added: I tested this using a virtual machine running Edge 44.17763.1.0 / EdgeHTML 18.17763, and can confirm that the white screen issue was present before making this change.

ngwese commented 4 years ago

...and sorry for the delay, I missed the GH notifications.