-
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
Fix gmf examples #3835
Fix gmf examples #3835
Conversation
ce208d5
to
5d3118d
Compare
a96c117
to
976a259
Compare
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.
See my comments and then please merge.
@@ -162,7 +162,7 @@ exports.Controller_.prototype.handleFeatureSet_ = function( | |||
keys.push( | |||
olEvents.listen( | |||
newFeature, | |||
olObject.getChangeEventType(propName), | |||
getChangeEventType(propName), |
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.
We should create the string ourselves: change:${propName}
.
See also the other files below where you should do for ex: 'change:dirty' and remove the imports.
src/map/FeatureOverlayMgr.js
Outdated
@@ -7,7 +7,7 @@ import * as olBase from 'ol/index.js'; | |||
import olLayerVector from 'ol/layer/Vector.js'; | |||
import * as olObj from 'ol/obj.js'; | |||
import olSourceVector from 'ol/source/Vector.js'; | |||
import olStyleStyle, {toFunction as toStyleFunction} from 'ol/style/Style.js'; | |||
import olStyleStyle, {toFunction as toStyleFunction, createDefaultStyle as defaultFunction} from 'ol/style/Style.js'; |
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.
Better call it olStyleDefaultFunction
.
Fix several gmf examples:
Fix ngeo example:
Also various fixes for gmf apps.