remove let

[graingert]

const is better than let, and you never need to use let - if you think functionally enough.

[LostCrew]

@graingert Ok like that, but I'm not a fan of the ternary operator, especially in long expressions. Midly relatedly, I identify Math.round(Math.max(this.$on.width(), this.$off.width())) possibly being a separate function.

[graingert]

We can replace the ternary with do expressions when they move to stage 4

…

On 25 Feb 2017 15:15, "Emanuele Marchi" ***@***.***> wrote: @graingert <https://github.com/graingert> Ok like that, but I'm not a fan of the ternary operator, especially in long expressions. Midly relatedly, I identify Math.round(Math.max(this.$on.width(), this.$off.width())) possibly being a separate function. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#619 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAZQTNctz2ylu5EZWMnRPgxSBvlsyaWsks5rgEV_gaJpZM4MC7YI> .

[LostCrew]

@graingert I'm adding stage-2 now, it might help