Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(110)

Unified Diff: STYLEGUIDE.md

Issue 2287683002: Add naming note to styleguide (Closed) Base URL: https://skia.googlesource.com/buildbot@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: STYLEGUIDE.md
diff --git a/STYLEGUIDE.md b/STYLEGUIDE.md
index 55d0b3a1e5604631f5461cac5e040544dc9fb8b2..f767afd3fedb4ee4d045ebe73c2f0593c246ed04 100644
--- a/STYLEGUIDE.md
+++ b/STYLEGUIDE.md
@@ -46,6 +46,17 @@ If an Element has a helper element that should not be used alone, it may be incl
in the same file.
Example: [details-summary.html](https://skia.googlesource.com/buildbot/+/d3624df97a7422c542a739f36668f4831a2cda0b/res/imp/details-summary.html)
+Naming Things
+-------------
+
+Properties should be named using one word (if possible) or using snake_case.
+Private properties and methods should be prefixed with an underscore
+(e.g. _super_private).
+
+**Rationale:** The names are consistent in Javascript and HTML land.
+Polymer will automatically convert camelCase into sausage-case otherwise, which
+leads to hilarity and/or tears.
+
Python Style Guide
==================
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698