|
|
Line 74: |
Line 74: |
| * [[Gaia/Weather|Weather]] | | * [[Gaia/Weather|Weather]] |
|
| |
|
| == Contributing ==
| | See [[Gaia/Hacking]] to get started. |
| === Filing Bugs ===
| |
| Issues are filed on [https://github.com/mozilla-b2g/gaia/issues Github].
| |
| | |
| === Hacking ===
| |
| You can fork us on [https://github.com/mozilla-b2g/gaia Github], then send a pull request.
| |
| | |
| A lot of the apps should be able to run in a web browser (particularly Firefox Nightly), but sometimes you may need to run them on B2G until new APIs land in browsers. See [[Gaia/Hacking]] to get started.
| |
| | |
| === Communicating ===
| |
| You can find us in #gaia on irc.mozilla.org and we use the [https://lists.mozilla.org/listinfo/dev-gaia dev-gaia mailing list].
| |
| | |
| === Reviewers ===
| |
| | |
| Gaia does not have formal module owners, but the following developers are good candidates to review patches to the following modules and apps:
| |
| | |
| {|border=1
| |
| !Module
| |
| !Reviewers (irc nick/github username)
| |
| |-
| |
| |Dialer
| |
| |etienne_s/etiennesegonzac
| |
| |-
| |
| |Messages
| |
| |timdream
| |
| |-
| |
| |Browser
| |
| |benfrancis/benfrancis
| |
| |-
| |
| |Camera
| |
| |benfrancis/benfrancis
| |
| |-
| |
| |Gallery
| |
| |djf/davidflanagan, benfrancis/benfrancis
| |
| |-
| |
| |Video
| |
| |djf/davidflanagan, kaze/fabi1cazenave
| |
| |-
| |
| |Homescreen
| |
| |djf/davidflanagan
| |
| |-
| |
| |Keyboard
| |
| |timdream
| |
| |-
| |
| |Settings
| |
| |kaze/fabi1cazenave
| |
| |-
| |
| |Calculator
| |
| |daleharvey/daleharvey
| |
| |-
| |
| |Music
| |
| |djf/davidflanagan
| |
| |-
| |
| |Clock
| |
| |etienne_s/etiennesegonzac
| |
| |}
| |
| | |
| === Coding Style ===
| |
| * Background:
| |
| ** [[MDC:Developer Guide/Coding Style#General practices]]
| |
| ** [[MDC:Developer Guide/Coding Style#JavaScript practices]]
| |
| ** [[MDC:Developer Guide/Coding Style#Naming and formatting code]]
| |
| | |
| * make sure HTML files are declared <!DOCTYPE html> (i.e., HTML5). IE9+ will load them in compatibility mode otherwise.
| |
| | |
| * add a <code>"use strict";</code> statement (exactly that!) to the top of your JS files
| |
| | |
| * 2 spaces for indentation - do not use tab.
| |
| | |
| * Line break are free (I promise) don't hesitate to use them to separate logical block inside your functions.
| |
| | |
| * Files are named <code>like_this.js</code>.
| |
| | |
| * Use single quote instead of double quotes.
| |
| | |
| * Additional rules:
| |
| Bad:
| |
| if (expression) doSomething();
| |
| | |
| Correct:
| |
| if (expression)
| |
| doSomething();
| |
| | |
| === Before submitting a patch ===
| |
| On each javascript files you are adding or you have modified, run:
| |
| gjslint --nojsdoc my_file.js
| |
| | |
| http://code.google.com/closure/utilities/docs/linter_howto.html
| |
| | |
| Set of smoke tests to run before you check-in a patch:
| |
| [[media:PatchSmokeTests.pdf|PatchSmokeTests.pdf]]
| |
|
| |
|
| == Subpages of {{FULLPAGENAME}}== | | == Subpages of {{FULLPAGENAME}}== |
| {{Special:PrefixIndex/{{FULLPAGENAME}}/}} | | {{Special:PrefixIndex/{{FULLPAGENAME}}/}} |