This PR is minimum level of refactoring. Here are some notes.
Separating mappanel.js and iopanel.js was difficult due to many dependencies, so I gave up to separate those and concatenate it as mapiopanel.js.
Basically, each <input> control related methods need to be defined in the panel which includes it, but currently, the methods are in most used panel or functionality. (ex. getArgumentValue in geosop.js)
Move html inside event handler to each panel's inside with addEventListener.
For future, something MVVM? or state management? may solve above problems, but I will think about it, after upgrading OpenLayers.
TODO:
[x] Currently, all operations are failing in geomToWkt method's geos.GEOSNormalize(geom) line. I need to fix this bug.
Closes #7
This PR is minimum level of refactoring. Here are some notes.
mappanel.js
andiopanel.js
was difficult due to many dependencies, so I gave up to separate those and concatenate it asmapiopanel.js
.<input>
control related methods need to be defined in the panel which includes it, but currently, the methods are in most used panel or functionality. (ex.getArgumentValue
ingeosop.js
)addEventListener
.For future, something MVVM? or state management? may solve above problems, but I will think about it, after upgrading OpenLayers.
TODO:
geomToWkt
method'sgeos.GEOSNormalize(geom)
line. I need to fix this bug.