[OpenLayers-Trac] Re: [OpenLayers] #2947: allow custom jsfiles array
OpenLayers
trac-20090302 at openlayers.org
Tue Dec 7 16:36:04 EST 2010
#2947: allow custom jsfiles array
---------------------+------------------------------------------------------
Reporter: erilem | Owner:
Type: feature | Status: new
Priority: minor | Milestone: 2.11 Release
Component: general | Version: 2.10
Keywords: | State: Review
---------------------+------------------------------------------------------
Comment(by erilem):
Replying to [comment:6 tschaub]:
> Thanks for making the change Eric. This is looking good. Your patch is
missing the OpenLayersJsFiles.html test page. Did you forget to add it?
I did. See [attachment:patch-2947-A2.diff].
I've been thinking... Wouldn't it actually be better (more natural) to be
able to do:
{{{
<script type="text/javascript"
src="../lib/OpenLayers.js?noautoload=true></script>
<script type="text/javascript"
src="../lib/OpenLayers/Util.js></script>
}}}
instead of
{{{
<script type="text/javascript">
var OpenLayers = [
"OpenLayers/Util.js"
];
</script>
<script type="text/javascript" src="../lib/OpenLayers.js></script>
}}}
With my patch the following can be done:
{{{
<script type="text/javascript">
var OpenLayers = [];
</script>
<script type="text/javascript" src="../lib/OpenLayers.js></script>
<script type="text/javascript"
src="../lib/OpenLayers/Util.js></script>
}}}
but this isn't user-friendly.
Tell me what you think. I'm happy to commit the current patch, or to
provide a new patch for the {{{noautoload}}} solution.
--
Ticket URL: <http://trac.openlayers.org/ticket/2947#comment:7>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer
More information about the Trac
mailing list