From c3426d6bb674263b2834d8547fcf4c407555dbcf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Iv=C3=A1n=20S=C3=A1nchez=20Ortega?= Date: Wed, 20 Apr 2016 09:38:58 +0200 Subject: [PATCH] docstring fixes regarding Polyline/Circle options --- src/layer/vector/Circle.js | 6 ++++-- src/layer/vector/CircleMarker.js | 6 ++++-- src/layer/vector/Path.js | 4 ++-- src/layer/vector/Polyline.js | 4 +++- 4 files changed, 13 insertions(+), 7 deletions(-) diff --git a/src/layer/vector/Circle.js b/src/layer/vector/Circle.js index 097cb53e..945484f8 100644 --- a/src/layer/vector/Circle.js +++ b/src/layer/vector/Circle.js @@ -26,6 +26,8 @@ L.Circle = L.CircleMarker.extend({ if (isNaN(this.options.radius)) { throw new Error('Circle radius cannot be NaN'); } + // @section + // @aka Circle options // @option radius: Number; Radius of the circle, in meters. this._mRadius = this.options.radius; }, @@ -91,11 +93,11 @@ L.Circle = L.CircleMarker.extend({ } }); -// @factory L.circle(latlng: LatLng, options?: Path options) +// @factory L.circle(latlng: LatLng, options?: Circle options) // Instantiates a circle object given a geographical point, and an options object // which contains the circle radius. // @alternative -// @factory L.circle(latlng: LatLng, radius: Number, options?: Path options) +// @factory L.circle(latlng: LatLng, radius: Number, options?: Circle options) // Obsolete way of instantiating a circle, for compatibility with 0.7.x code. // Do not use in new applications or plugins. L.circle = function (latlng, options, legacyOptions) { diff --git a/src/layer/vector/CircleMarker.js b/src/layer/vector/CircleMarker.js index 07c81043..7690247b 100644 --- a/src/layer/vector/CircleMarker.js +++ b/src/layer/vector/CircleMarker.js @@ -8,6 +8,8 @@ L.CircleMarker = L.Path.extend({ + // @section + // @aka CircleMarker options options: { fill: true, @@ -85,8 +87,8 @@ L.CircleMarker = L.Path.extend({ }); -// @factory L.circleMarker(latlng: LatLng, options? CircleMarker options) -// +// @factory L.circleMarker(latlng: LatLng, options?: CircleMarker options) +// Instantiates a circle marker object given a geographical point, and an optional options object. L.circleMarker = function (latlng, options) { return new L.CircleMarker(latlng, options); }; diff --git a/src/layer/vector/Path.js b/src/layer/vector/Path.js index 90b992fb..d9cbb623 100644 --- a/src/layer/vector/Path.js +++ b/src/layer/vector/Path.js @@ -37,11 +37,11 @@ L.Path = L.Layer.extend({ lineJoin: 'round', // @option dashArray: String = null - // A string that defines the stroke [dash pattern](https://developer.mozilla.org/docs/Web/SVG/Attribute/stroke-dasharray). Doesn't work on canvas-powered layers (e.g. Android 2). + // A string that defines the stroke [dash pattern](https://developer.mozilla.org/docs/Web/SVG/Attribute/stroke-dasharray). Doesn't work on `Canvas`-powered layers in [some old browsers](https://developer.mozilla.org/docs/Web/API/CanvasRenderingContext2D/setLineDash#Browser_compatibility). dashArray: null, // @option dashOffset: String = null - // A string that defines the [distance into the dash pattern to start the dash](https://developer.mozilla.org/docs/Web/SVG/Attribute/stroke-dashoffset). Doesn't work on canvas-powered layers + // A string that defines the [distance into the dash pattern to start the dash](https://developer.mozilla.org/docs/Web/SVG/Attribute/stroke-dashoffset). Doesn't work on `Canvas`-powered layers in [some old browsers](https://developer.mozilla.org/docs/Web/API/CanvasRenderingContext2D/setLineDash#Browser_compatibility). dashOffset: null, // @option fill: Boolean = depends diff --git a/src/layer/vector/Polyline.js b/src/layer/vector/Polyline.js index be79a185..346b6a4d 100644 --- a/src/layer/vector/Polyline.js +++ b/src/layer/vector/Polyline.js @@ -38,6 +38,8 @@ L.Polyline = L.Path.extend({ + // @section + // @aka Polyline options options: { // @option smoothFactor: Number = 1.0 // How much to simplify the polyline on each zoom level. More means @@ -275,7 +277,7 @@ L.Polyline = L.Path.extend({ } }); -// @factory L.polyline(latlngs: LatLng[], options?: Path options) +// @factory L.polyline(latlngs: LatLng[], options?: Polyline options) // Instantiates a polyline object given an array of geographical points and // optionally an options object. You can create a `Polyline` object with // multiple separate lines (`MultiPolyline`) by passing an array of arrays