mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-15 10:35:48 +00:00
4854d644ee
Us grouping the inheritable static properties that way is an implementation detail that is polluting the index and makes it harder to refer to individual identifiers. It also causes problems under JSDuck 5 because that version is more strict about defining properties (Foo.static.bar) of which the parent is not defined in the index (Foo.static), we'd have to add a sea of `@static @property {Object} this.static` all over the place. Might as well hide this implementation detail and just consider them static properties (just like we already do for "private" properties). Change-Id: Ibf2ebf7752aabc2b75b6ac6fa00e2284a181a600
181 lines
5.2 KiB
Markdown
181 lines
5.2 KiB
Markdown
# Contributing to VisualEditor
|
|
|
|
Thank you for helping us develop VisualEditor!
|
|
|
|
This document describes how to report bugs, set up your development
|
|
environment, run tests, and build documentation. It also provides the coding
|
|
conventions we use in the project.
|
|
|
|
## Bug reports
|
|
|
|
Please report bugs to [bugzilla.wikimedia.org](https://bugzilla.wikimedia.org/enter_bug.cgi?product=VisualEditor&component=General)
|
|
using the `VisualEditor` product. Feel free to use the `General`
|
|
component if you don't know where else your bug might belong. Don't
|
|
worry about specifying version, severity, hardware, or OS.
|
|
|
|
## Running tests
|
|
|
|
VisualEditor's build scripts use the [Grunt](http://gruntjs.com/) task runner.
|
|
To install it make sure you have [node and npm](http://nodejs.org/download/)
|
|
installed, then run:
|
|
|
|
```sh
|
|
# Install Grunt command-line utility
|
|
$ npm install -g grunt-cli
|
|
|
|
# Install VisualEditor's dev dependencies
|
|
$ npm install
|
|
```
|
|
|
|
To run the tests, use:
|
|
```sh
|
|
$ grunt test
|
|
```
|
|
|
|
For other grunt tasks, see:
|
|
```sh
|
|
$ grunt --help
|
|
```
|
|
|
|
To run the tests in a web browser, make sure your MediaWiki install is
|
|
[configured](https://www.mediawiki.org/wiki/Manual:JavaScript_unit_testing) to
|
|
allow running of tests. Set in `LocalSettings.php`:
|
|
```php
|
|
// https://www.mediawiki.org/wiki/Manual:JavaScript_unit_testing
|
|
$wgEnableJavaScriptTest = true;
|
|
```
|
|
|
|
Then open `http://URL_OF_MEDIAWIKI/index.php/Special:JavaScriptTest/qunit`
|
|
(for example, <http://localhost/w/index.php/Special:JavaScriptTest/qunit>).
|
|
|
|
## Building documentation
|
|
|
|
VisualEditor uses [JSDuck](https://github.com/senchalabs/jsduck) to process
|
|
documentation comments embedded in the code. To build the documentation, you
|
|
will need `ruby`, `gem`, and `jsduck` installed.
|
|
|
|
### Installing ruby and gem
|
|
|
|
You're mostly on your own here, but we can give some hints for Mac OS X.
|
|
|
|
##### Installing Gem in Mac OS X
|
|
Ruby ships with OSX but may be outdated. Use [Homebrew](http://mxcl.github.com/homebrew/):
|
|
```sh
|
|
$ brew install ruby
|
|
```
|
|
|
|
If you've never used `gem` before, don't forget to add the gem's bin to your
|
|
`PATH` ([howto](http://stackoverflow.com/a/14138490/319266)).
|
|
|
|
### Installing jsduck
|
|
|
|
Once you have gem, installing [JSDuck](https://github.com/senchalabs/jsduck) is easy:
|
|
```sh
|
|
$ gem install --user-install jsduck
|
|
```
|
|
|
|
### Running jsduck
|
|
|
|
Creating the documentation is easy:
|
|
```sh
|
|
$ cd VisualEditor
|
|
$ .docs/generate.sh
|
|
```
|
|
|
|
You may need to set `MW_INSTALL_PATH` in your environment to the location of
|
|
your mediawiki installation if VisualEditor is not checked out directly in the
|
|
mediawiki extensions folder (for example, if you're using a symlink).
|
|
|
|
The generated documentation is in the `docs/` subdirectory. View the
|
|
documentation at
|
|
`http://URL_OF_MEDIAWIKI/extensions/VisualEditor/docs/`
|
|
(for example, <http://localhost/w/extensions/VisualEditor/docs>).
|
|
|
|
Note that `jsduck` doesn't support browsing vis the `file:` protocol.
|
|
|
|
## VisualEditor Code Guidelines
|
|
|
|
We inherit the code structure (about whitespace, naming and comments) conventions
|
|
from MediaWiki. See [Manual:Coding conventions/JavaScript](https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript)
|
|
on mediawiki.org.
|
|
|
|
Git commit messages should follow the conventions described in
|
|
<https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines>.
|
|
|
|
### Documentation comments
|
|
|
|
* End sentences in a full stop.
|
|
* Continue sentences belonging to an annotation on the next line, indented with an
|
|
additional space.
|
|
* Types in documentation comments should be separated by a pipe character. Use types
|
|
that are listed in the Types section of this document, otherwise use the identifier
|
|
(full path from the global scope) of the constructor function (e.g. `{ve.dm.BranchNode}`).
|
|
|
|
### Annotations
|
|
|
|
We use the following annotations. They should be used in the order as they are described
|
|
here, for consistency. See [JSDuck/Tags](https://github.com/senchalabs/jsduck/wiki/Tags) for more elaborate documentation.
|
|
|
|
* @class Name (optional, guessed)
|
|
* @abstract
|
|
* @extends ClassName
|
|
* @mixins ClassName
|
|
* @constructor
|
|
* @private
|
|
* @static
|
|
* @method name (optional, guessed)
|
|
* @template
|
|
* @property name (optional, guessed)
|
|
* @until Text: Optional text.
|
|
* @source Text
|
|
* @context {Type} Optional text.
|
|
* @inheritable
|
|
* @param {Type} name Optional text.
|
|
* @emits name
|
|
* @returns {Type} Optional text.
|
|
* @chainable
|
|
* @throws {Type}
|
|
|
|
### Types
|
|
|
|
Special values:
|
|
* undefined
|
|
* null
|
|
* this
|
|
|
|
Primitive types:
|
|
* boolean
|
|
* number
|
|
* string
|
|
|
|
Built-in classes:
|
|
* Array
|
|
* Date
|
|
* Function
|
|
* RegExp
|
|
* Object
|
|
|
|
Browser classes:
|
|
* HTMLElement
|
|
|
|
jQuery classes:
|
|
* jQuery
|
|
* jQuery.Event
|
|
|
|
## Add a new javascript class
|
|
|
|
When a new javascript class is added, the file must be referenced in a number of places
|
|
before it can be used.
|
|
|
|
Test files:
|
|
* VisualEditor.hooks.php in onResourceLoaderTestModules
|
|
|
|
Regular files:
|
|
* .docs/categories.json in General->Utilities (or somewhere more specific)
|
|
* VisualEditor.php in ext.visualEditor.core (or somewhere more specific)
|
|
* Run `php maintenance/makeStaticLoader.php --target demo --write-file demos/ve/index.php`
|
|
* Run `php maintenance/makeStaticLoader.php --target test --write-file modules/ve/test/index.php`
|
|
|
|
makeStaticLoader.php is a maintenance script to automatically generate an HTML document fragment
|
|
containing script tags in dependency order (for standalone environments without ResourceLoader).
|