Well, when the code looks great, sometimes there's nearly nothing to suggest.
On my answer, a user comments that I should not suggest formatting changes unless they damage readability or cause confusion.
As long as everyone involved in the maintenance of this code agrees that this is how they want it formatted, then there is no mistake. Nitpick about formatting when its inconsistent or significantly impacts readability, not because its in a style that differs from your own.
Is it always ok to suggest best formatting practices on code that doesn't follow popular conventions? Or should I not submit these formatting improvement suggestions?
varis not a style thing, it's just plain wrong. It won't work in strict mode or ES6 and there is absolutely never a reason to do it, outside of extreme edge cases like code that's going to get combined with some other code at build time where the variable is declared properly. \$\endgroup\$