[OpenLayers-Trac] Re: [OpenLayers] #1099: deprecate OpenLayers.Element hide() / show()

OpenLayers trac-20090302 at openlayers.org
Sun Feb 27 16:06:30 EST 2011


#1099: deprecate OpenLayers.Element hide() / show()
-------------------------------+--------------------------------------------
 Reporter:  euzuro             |       Owner:  euzuro      
     Type:  feature            |      Status:  new         
 Priority:  minor              |   Milestone:  2.11 Release
Component:  BaseTypes.Element  |     Version:  2.5         
 Keywords:                     |       State:  Commit      
-------------------------------+--------------------------------------------
Changes (by crschmidt):

  * state:  Review => Commit


Comment:

 bartvde: Why have a function call defined with its corresponding overhead
 for something that we only use one place in the entire library? I don't
 think it's worth it to have these functions (or hook into another library
 for them) given how rarely we use them now. I think "Explicit is better
 than implicit" is true here.

 I'm comfortable with fredj's patch here. If people want to have more
 discussion, that's fine, but given how little there is here, I don't think
 it's worth it :)

-- 
Ticket URL: <http://trac.openlayers.org/ticket/1099#comment:9>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list