-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add raster layers unit #4209
Add raster layers unit #4209
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, now, we can't choose anymore the number of digit ?
Effectively I will add this possibility :-) |
8bfcbdf
to
ef58f49
Compare
contribs/gmf/src/raster/component.js
Outdated
* </span> | ||
* | ||
* For value in meter `elevationLayersConfig` can be en empty object, complex example: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be en empty object -> an
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks :-)
@@ -60,6 +60,12 @@ const exports = function($scope, $injector) { | |||
*/ | |||
this.elevationLayers = ['aster', 'srtm']; | |||
|
|||
/** | |||
* @type {Array.<gmf.mobile.measure.pointComponent.LayerConfig>} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong type
contribs/gmf/src/raster/component.js
Outdated
* | ||
* @htmlAttribute {boolean} gmf-elevation-active A boolean to set active or | ||
* deactive the component. | ||
* @htmlAttribute {number} gmf-elevation-elevation The value to set with the | ||
* elevation value. | ||
* @htmlAttribute {string} gmf-elevation-layer Elevation layer to use. | ||
* @htmlAttribute {Array.<gmf.mobile.measure.pointComponent.LayerConfig>} gmf-elevation-layerconfig Elevation layer configurations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong type
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gmf-elevation-layersconfig
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks :-)
contribs/gmf/src/raster/component.js
Outdated
@@ -132,13 +153,19 @@ exports.Controller_ = function($scope, ngeoDebounce, gmfRaster) { | |||
*/ | |||
this.gmfRaster_ = gmfRaster; | |||
|
|||
/** | |||
* @type {!Array.<gmf.mobile.measure.pointComponent.LayerConfig>} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong type
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed :-)
contribs/gmf/src/raster/component.js
Outdated
@@ -313,6 +354,12 @@ exports.WidgetController_ = function() { | |||
*/ | |||
this.layers; | |||
|
|||
/** | |||
* @type {!Array.<gmf.mobile.measure.pointComponent.LayerConfig>} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong type
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed :-)
* separator: (string|undefined) | ||
* }} | ||
*/ | ||
exports.LayerConfig; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
layersConfig
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just for one layer ..., I use it as Object.<string, gmf.raster.component.LayerConfig>
for layers config :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's right, sorry
ef58f49
to
d01d8d3
Compare
d01d8d3
to
677d36f
Compare
fix https://jira.camptocamp.com/browse/GSGMF-620