https://github.com/phetsims/chipper/issues/728

restore checklist item for toFixed
phetsims · Jan 31, 2019 · 32926ac · 32926ac
1 parent 546ff36
commit 32926ac
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/checklists/code_review_checklist.md b/checklists/code_review_checklist.md
@@ -454,6 +454,10 @@ addListener: function( listener ) { /*...*/ }
 * Regex for property assignment like `x.y = something`: `[\w]+\.[\w]+\s=`
 * Regex for function declarations: `[\w]+: function\(`
 
+#### **Math Libraries**
+
+- [ ] `DOT/Util.toFixed` or `DOT/Util.toFixedNumber` should be used instead of `toFixed`. JavaScript's `toFixed` is notoriously buggy. Behavior differs depending on browser, because the spec doesn't specify whether to round or floor.
+
 #### IE11
 - [ ] No usage of `Number.parseInt()`
 - [ ] No usage of `Array.prototype.find`