Unified Diff: pages/coding-style.html
Issue 29386587:
Issue 4993 - Add operator-linebreak rule to our JavaScript style guide (Closed)
Patch Set:
Created March 17, 2017, 10:42 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: pages/coding-style.html |
| diff --git a/pages/coding-style.html b/pages/coding-style.html |
| index a2791736e26db9a375cbcc7185985d1fa73ebaee..900f15e0122ebc45a0e14cd97e61f5ac610afa79 100644 |
| --- a/pages/coding-style.html |
| +++ b/pages/coding-style.html |
| @@ -25,6 +25,7 @@ title=Coding Style |
| <ul> |
| <li>{{s15 Follow the Mozilla Coding Style's <a href="https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style#JavaScript_practices">JavaScript practices</a>.}}</li> |
| <li>{{javascript-strict Always use <a href="https://developer.mozilla.org/docs/Web/JavaScript/Reference/Strict_mode">strict mode</a>.}}</li> |
| + <li>{{javascript-operator-linebreak When splitting overlong expressions into multiple lines the joining operators should always be left at the end of the line, before the linebreak.}}</li> |
|
Wladimir Palant
2017/03/17 11:35:02
"the joining binary operators" please. Also, we ne
kzar
2017/03/17 12:58:01
Done.
|
| <li>{{javascript-if-else-braces When an <code><fix>if</fix></code> statement, an <code><fix>else</fix></code> statement or a loop spans over more than one line always enclose it with braces.}}</li> |
| <li>{{s18 In classes, prefix private functions with a single underscore to make them pseudo-private.}}</li> |
| <li>{{javascript-spacing-literals Don't add any spaces immediately after opening, or before closing, an object or array literal.}}</li> |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
