search

microsoft / TypeScriptSamples

Community Driven Samples for TypeScript
3.19k stars 1.78k forks source link

Update raytracer.ts #147

Closed gabrielsroka closed 5 years ago

gabrielsroka commented 6 years ago

ctx.fillRect() last 2 args should be width and height (not x2, y2) https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/fillRect

will this fix https://www.typescriptlang.org/examples/raytracer.ts which is used in https://www.typescriptlang.org/play/index.html too?

orta commented 5 years ago

👍 thanks!

orta commented 5 years ago

This won't update that template, but I have sent a PR for that internally