Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

docs: fix grammar and spelling mistakes #895

Merged
merged 1 commit into from
Sep 15, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion DEVELOPER.md
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ chmod u+x .git/hooks/pre-commit
Webdriver Test
--------------

Now `zone.js` also support run webdriver e2e test.
`zone.js` also supports running webdriver e2e tests.

1. run locally

Expand Down
13 changes: 6 additions & 7 deletions MODULE.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
# Modules

zone.js patches the async APIs which described above, but those patch will have some overhead,
from zone.js v0.8.9, you can choose which web API module you want to patch, for example,
the below samples show how to disable some modules, you just need to define some global variables
before load zone.js.
Starting from zone.js v0.8.9, you can choose which web API modules you want to patch as to reduce overhead introduced by the patching of these modules. For example,
the below samples show how to disable some modules. You just need to define a few global variables
before loading zone.js.

```
<script>
Expand All @@ -17,7 +16,7 @@ before load zone.js.
<script src="../dist/zone.js"></script>
```

Below is the full list of current support modules.
Below is the full list of currently supported modules.

- Common

Expand Down Expand Up @@ -58,7 +57,7 @@ Below is the full list of current support modules.

- on_property

you can also disable specified on_property by setting `__Zone_ignore_on_properties`, for example,
You can also disable specific on_properties by setting `__Zone_ignore_on_properties` as follows: for example,
if you want to disable `window.onmessage` and `HTMLElement.prototype.onclick` from zone.js patching,
you can do like this.

Expand All @@ -79,7 +78,7 @@ you can do like this.

- Angular(2+)

Angular use zone.js to manage async operations and decide when to perform change detection, so in Angular,
Angular uses zone.js to manage async operations and decide when to perform change detection. Thus, in Angular,
the following APIs should be patched, otherwise Angular may not work as expected.

1. ZoneAwarePromise
Expand Down
19 changes: 9 additions & 10 deletions NON-STANDARD-APIS.md
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
# Zone.js's support for non standard apis

Zone.js patched most standard APIs so they can be in zone. such as DOM events listener, XMLHttpRequest in Browser
and EventEmitter, fs API in nodejs.
Zone.js patched most standard APIs such as DOM event listeners, XMLHttpRequest in Browser, EventEmitter and fs API in Node.js so they can be in zone.

But there are still a lot of non standard APIs are not patched by default, such as MediaQuery, Notification,
WebAudio and so on. We are adding the support to those APIs, and the progress will be updated here.
But there are still a lot of non standard APIs that are not patched by default, such as MediaQuery, Notification,
WebAudio and so on. We are adding support to those APIs, and our progress is updated here.

## Currently supported non standard Web APIs

Expand Down Expand Up @@ -95,9 +94,9 @@ to load the patch, you should load in the following order.
## Usage

By default, those APIs' support will not be loaded in zone.js or zone-node.js,
so if you want to load those API's support, you should load those files by yourself
so if you want to load those API's support, you should load those files by yourself.

for example, if you want to add MediaQuery patch, you should do like this.
For example, if you want to add MediaQuery patch, you should do like this:

```
<script src="path/zone.js"></script>
Expand All @@ -107,7 +106,7 @@ for example, if you want to add MediaQuery patch, you should do like this.
* rxjs

`zone.js` also provide a `rxjs` patch to make sure rxjs Observable/Subscription/Operator run in correct zone.
for detail please refer to [pull request 843](https://github.com/angular/zone.js/pull/843), the following sample code describe the idea.
For details please refer to [pull request 843](https://github.com/angular/zone.js/pull/843). The following sample code describes the idea.

```
const constructorZone = Zone.current.fork({name: 'constructor'});
Expand Down Expand Up @@ -143,19 +142,19 @@ operatorZone.run(() => {
});
```

currently basically all `rxjs` API include
Currently basically everything the `rxjs` API includes

- Observable
- Subscription
- Subscriber
- Operators
- Scheduler

are patched, so they will run in the correct zone.
is patched, so each asynchronous call will run in the correct zone.

## Usage.

for example, in angular application, you can load this patch in your `app.module.ts`.
For example, in an Angular application, you can load this patch in your `app.module.ts`.

```
import 'zone.js/dist/zone-patch-rxjs';
Expand Down
14 changes: 7 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ Implements _Zones_ for JavaScript, inspired by [Dart](https://www.dartlang.org/a

> If you're using zone.js via unpkg please provide a query param `?main=browser`
`https://unpkg.com/zone.js?main=browser`
> If you're using the following library, make sure you import them first
> If you're using any of the following libraries, make sure you import them first

> * 'newrelic' as it patches global.Promise before zone.js does
> * 'async-listener' as it patches global.setTimeout, global.setInterval before zone.js does
Expand Down Expand Up @@ -37,19 +37,19 @@ See this video from ng-conf 2014 for a detailed explanation:
## Standard API support

zone.js patched most standard web APIs(such as DOM events, XMLHttpRequest...) and nodejs APIs
(EventEmitter,fs ...), for more details, please see [STANDARD-APIS.md](STANDARD-APIS.md) for
more details.
(EventEmitter,fs ...), for more details, please see [STANDARD-APIS.md](STANDARD-APIS.md).

## Non standard API support

And now we are adding support to some non standard APIs, such as MediaQuery,
We are adding support to some non standard APIs, such as MediaQuery and
Notification. Please see [NON-STANDARD-APIS.md](NON-STANDARD-APIS.md) for more details.

## Modules

zone.js patches the async APIs which described above, but those patch will have some overhead,
from zone.js v0.8.9, you can choose which web API module you want to patch, for detail, please
see [MODULE.md](MODULE.md) for more details.
zone.js patches the async APIs described above, but those patches will have some overhead.
Starting from zone.js v0.8.9, you can choose which web API module you want to patch.
For more details, please
see [MODULE.md](MODULE.md).

## Promise A+ test passed
[![Promises/A+ 1.1 compliant](https://promisesaplus.com/assets/logo-small.png)](https://promisesaplus.com/)
Expand Down
51 changes: 24 additions & 27 deletions STANDARD-APIS.md
Original file line number Diff line number Diff line change
@@ -1,24 +1,22 @@
# Zone.js's support for standard apis

Zone.js patched most standard APIs so they can be in zone. such as DOM events listener, XMLHttpRequest in Browser
and EventEmitter, fs API in nodejs.
Zone.js patched most standard APIs such as DOM event listeners, XMLHttpRequest in Browser, EventEmitter and fs API in Node.js so they can be in zone.

In this document, all patched API will be listed here.
In this document, all patched API are listed.

for non standard API, please reference to [NON-STANDARD-APIS.md](NON-STANDARD-APIS.md)
For non-standard APIs, please see [NON-STANDARD-APIS.md](NON-STANDARD-APIS.md)

## Patch Mechanism
## Patch Mechanisms

There are several patch mechanism
There are several patch mechanisms

- wrap, wrap will make callback in zone, and application can receive onInvoke, onIntercept callback
- Task, just like javascript vm, application can receive onScheduleTask, onInvokeTask, onCancelTask, onHasTask callback
- wrap: makes callbacks run in zones, and makes applications able to receive onInvoke and onIntercept callbacks
- Task: just like in the JavaScript VM, applications can receive onScheduleTask, onInvokeTask, onCancelTask and onHasTask callbacks
1. MacroTask
2. MicroTask
2. MicroTask
3. EventTask

Now there are some APIs should be treated as Tasks, but still be patched in wrap way, they will be
patched as Task soon.
Some APIs which should be treated as Tasks, but are currently still patched in the wrap way. These will be patched as Tasks soon.

## Browser

Expand Down Expand Up @@ -54,9 +52,9 @@ Web APIs

EventTarget

- For browser support eventTarget, zone.js just patch EventTarget, so everything
inherit from EventTarget will also be patched.
- For browser does not support eventTarget, zone.js will patch the following APIs in the IDL
- For browsers supporting EventTarget, Zone.js just patches EventTarget, so everything that inherits
from EventTarget will also be patched.
- For browsers that do not support EventTarget, Zone.js will patch the following APIs in the IDL
that inherit from EventTarget

|||||
Expand All @@ -69,8 +67,7 @@ inherit from EventTarget will also be patched.
|IDBRequest|IDBOpenDBRequest|IDBDatabase|IDBTransaction|
|IDBCursor|DBIndex|WebSocket|

on properties, such as onclick, onreadystatechange, the following on properties will
be patched as EventTask, the following is the on properties zone.js patched
The following 'on' properties, such as onclick, onreadystatechange, are patched in Zone.js as EventTasks

|||||
|---|---|---|---|
Expand Down Expand Up @@ -104,28 +101,28 @@ be patched as EventTask, the following is the on properties zone.js patched
| crypto | MacroTask | |
| fs | MacroTask | all async methods are patched |

EventEmitter, addEventListener, prependEventListener, on, once will be patched as EventTask, and removeEventListener,
removeAllListeners will remove those eventTasks
EventEmitter, addEventListener, prependEventListener and 'on' will be patched once as EventTasks, and removeEventListener and
removeAllListeners will remove those EventTasks

## Electron

zone.js did not patch electron API, but in electron, both browser APIs and node APIs are patched, so
when you want to include zone.js in Electron, please use dist/zone-mix.js
Zone.js does not patch the Electron API, although in Electron both browser APIs and node APIs are patched, so
if you want to include Zone.js in Electron, please use dist/zone-mix.js

## ZoneAwareError

ZoneAwareError replace global Error, and it will add zone information to stack trace.
ZoneAwareError will also handle 'this' issue.
such as create an error without new, then this will be undefined in strict mode, and global in
non-strict mode. it will cause some issues and very difficult to detect.
ZoneAwareError replaces global Error, and adds zone information to stack trace.
ZoneAwareError also handles 'this' issue.
This type of issue would happen when creating an error without `new`: `this` would be `undefined` in strict mode, and `global` in
non-strict mode. It could cause some very difficult to detect issues.

```javascript
const error = Error();
```

ZoneAwareError will make sure that `this` is ZoneAwareError even without new.
ZoneAwareError makes sure that `this` is ZoneAwareError even without new.

## ZoneAwarePromise

ZoneAwarePromise wrap the global Promise and make it in zone and run as microTask,
it also passes promise A+ tests.
ZoneAwarePromise wraps the global Promise and makes it run in zones as a MicroTask.
It also passes promise A+ tests.
18 changes: 9 additions & 9 deletions lib/closure/zone_externs.js
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ var ZoneSpec = function() {};
ZoneSpec.prototype.name;

/**
* @type {Object<string, Object>|undefined} A set of properties to be associated with Zone. Use [Zone.get] to retrive them.
* @type {Object<string, Object>|undefined} A set of properties to be associated with Zone. Use [Zone.get] to retrieve them.
*/
ZoneSpec.prototype.properties;

Expand Down Expand Up @@ -266,21 +266,21 @@ var ZoneDelegate = function() {};
*/
ZoneDelegate.prototype.zone;
/**
* @param {!Zone} targetZone the [zone] which originally received the request.
* @param {!Zone} targetZone the [Zone] which originally received the request.
* @param {!ZoneSpec} zoneSpec the argument passed into the `fork` method.
* @returns {!Zone} the new forked zone
*/
ZoneDelegate.prototype.fork = function(targetZone, zoneSpec) {};
/**
* @param {!Zone} targetZone the [zone] which originally received the request.
* @param {!Zone} targetZone the [Zone] which originally received the request.
* @param {!Function} callback the callback function passed into `wrap` function
* @param {string=} source the argument passed into the `warp` method.
* @param {string=} source the argument passed into the `wrap` method.
* @returns {!Function}
*/
ZoneDelegate.prototype.intercept = function(targetZone, callback, source) {};

/**
* @param {Zone} targetZone the [zone] which originally received the request.
* @param {Zone} targetZone the [Zone] which originally received the request.
* @param {!Function} callback the callback which will be invoked.
* @param {?Object=} applyThis the argument passed into the `run` method.
* @param {?Array=} applyArgs the argument passed into the `run` method.
Expand All @@ -289,14 +289,14 @@ ZoneDelegate.prototype.intercept = function(targetZone, callback, source) {};
*/
ZoneDelegate.prototype.invoke = function(targetZone, callback, applyThis, applyArgs, source) {};
/**
* @param {!Zone} targetZone the [zone] which originally received the request.
* @param {!Object} error the argument passed into the `handleerror` method.
* @param {!Zone} targetZone the [Zone] which originally received the request.
* @param {!Object} error the argument passed into the `handleError` method.
* @returns {boolean}
*/
ZoneDelegate.prototype.handleError = function(targetZone, error) {};
/**
* @param {!Zone} targetZone the [zone] which originally received the request.
* @param {!Task} task the argument passed into the `scheduletask` method.
* @param {!Zone} targetZone the [Zone] which originally received the request.
* @param {!Task} task the argument passed into the `scheduleTask` method.
* @returns {!Task} task
*/
ZoneDelegate.prototype.scheduleTask = function(targetZone, task) {};
Expand Down
Loading