[OpenLayers-Users] line modify crashes safari

[Andreas Hocevar]

Hi,

thanks for your report Chris.

Christopher Schmidt wrote:
> When opening snapping.html, I see no geometries anymore.

Are you sure? I tried with all webkit browsers available to me (Safari 
on Windows, Epiphany-webkit and Chromium on Linux) and the geometries 
look just fine.

/me really hopes that this was just due to remainders of unpatched code 
in the browser cache or something, otherwise I have no idea for further 
workarounds because I cannot reproduce this.

> When drawing a geometry, I see:
> 
> TypeError: Result of expression 'target.parentNode' [null] is not an object.
> 
> many times as I drag along a snapping edge.

I was able to fix this one and attached a new patch to the ticket.

> Additionally, in the webkit ticket, you mentioned 
> 
>   http://dev.openlayers.org/releases/OpenLayers-2.8/examples/graphic-name.html
> 
> I think you meant:
> 
>   http://dev.openlayers.org/releases/OpenLayers-2.8/examples/snapping.html

Thanks, I fixed that. And looking at the bug history in webkit, it seems 
that the issue with the defs/use symbols not being rendered at the 
correct position and the crashing browser are two different issues, 
because crashes are not mentioned in 
https://bugs.webkit.org/show_bug.cgi?id=18994. It would be great if 
anyone could create a stripped down test case that shows the issue.

-Andreas.

> 
> Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_1; en-us) AppleWebKit/531.9 (KHTML, like Gecko) Version/4.0.3 Safari/531.9
> 
> is the version string.
> 
> -- Chris
> 
>> I also opened a webkit bug for the issue, see the ticket mentioned above
>> for a link.
>>
>> Regards,
>> Andreas.
>>
>> On 2010-01-07 03:12, Christopher Schmidt wrote:
>>> On Wed, Jan 06, 2010 at 11:43:37PM +0000, Wendell Turner wrote:
>>>   
>>>> Modifying a drawn feature on this example crashes the Safari
>>>> browser on both Macs and Windows.
>>>>
>>>>   http://dev.openlayers.org/releases/OpenLayers-2.8/examples/snapping.html
>>>>
>>>> Does OpenLayers intend to support Safari for this?
>>>>     
>>> There isn't a deliberate decision not to support Safari here.
>>>
>>> I feel like I played with this example in Safari 3 and it worked; however,
>>> I'm now running Safari4, and I can confirm that it crashes.
>>>
>>> The fact that it crashes the browser is obviously a browser bug,
>>> and should probably be reported as such. However, that doesn't mean
>>> that we shouldn't try to improve the behavior in OpenLayers; patches welcome.
>>>
>>> Regards,
>>>   
>>
>> -- 
>> Andreas Hocevar
>> OpenGeo - http://opengeo.org/
>> Expert service straight from the developers.
>>
> 


-- 
Andreas Hocevar
OpenGeo - http://opengeo.org/
Expert service straight from the developers.