TarVK / pixi-shadows

Adds dynamic shadows to PixiJS
https://tarvk.github.io/pixi-shadows/dist/
MIT License
53 stars 14 forks source link

tighten API #13

Open amir-arad opened 2 years ago

amir-arad commented 2 years ago

I just used _ as a convention to mark private fields. Now that TS is used, we can properly use the private keyword I think? I guess this is used by other parts of our code outside this class though, in which case we can add a public getter for this field I think.

Same comment applies to basically all occurrences of fields with _ at the start. Also for all methods with __ at the start, if they indeed aren't used outside the class at least.

_Originally posted by @TarVK in https://github.com/TarVK/pixi-shadows/pull/10#discussion_r912748394_