good looking out, bart. I've opened a ticket http://trac.openlayers.org/ticket/1696
if you feel like making the patch, i'll approve it. otherwise, i can make it when i get some freetime erik On Fri, Aug 22, 2008 at 11:42 AM, Bart van den Eijnden (OSGIS) <[EMAIL PROTECTED]> wrote: > Hi Erik, > > I haven't been using the Text layer that much lately to be honest, but this > changeset looks to have fixed that issue. > > Though if I read the code correctly, it seems that loadstart is triggered > twice? First in moveTo and then in loadText? This was fixed for GeoRSS [1], > but maybe not for Text. > > [1] http://trac.openlayers.org/ticket/1580 > > Best regards, > Bart > > Erik Uzureau wrote: >> >> Bart, Chris, >> >> I just looked in the code in trunk and this no longer seems to be the >> case.... >> Looks like this changeset got it: >> http://trac.openlayers.org/changeset/5828#file2 >> >> Can you confirm? >> >> Erik >> >> On Wed, Dec 19, 2007 at 9:54 AM, Christopher Schmidt >> <[EMAIL PROTECTED]> wrote: >> >>> >>> On Wed, Dec 19, 2007 at 09:51:25AM +0100, Bart van den Eijnden (OSGIS) >>> wrote: >>> >>>> >>>> Hi list, >>>> >>>> with the current code it seems impossible to listen to a loadstart event >>>> on >>>> the Text Layer, since it already fires this event in the constructor. >>>> >>>> Or am I missing something? >>>> >>> >>> Nope. That's a bug. The load should not actually be started in >>> initialize: instead, it should be started in moveTo, if it's not yet >>> loaded. Layer.GML does this, Layer.GeoRSS and Layer.Text do not. A patch >>> for this would be great. Please open a ticket -- if you plan to work on >>> it, or work with someone else on it, pleaes put it in 2.6 and I'll >>> review when it's completed. >>> >>> Regards, >>> -- >>> Christopher Schmidt >>> MetaCarta >>> _______________________________________________ >>> Dev mailing list >>> [email protected] >>> http://openlayers.org/mailman/listinfo/dev >>> >>> >> >> >> > > > -- > Bart van den Eijnden > OSGIS, Open Source GIS > [EMAIL PROTECTED] > http://www.osgis.nl > > _______________________________________________ Dev mailing list [email protected] http://openlayers.org/mailman/listinfo/dev
