[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