Letractively / svgweb

Automatically exported from code.google.com/p/svgweb
Other
0 stars 0 forks source link

Opera throws exception on patch to currentTranslate #358

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
The following population pyramid uses SVG Web r934
http://vis.uell.net/gsvg/poppyr11a.html

Opera browsers in their standard configuration (identifying as Opera) only 
partially render the 
population pyramid. The static grid gets painted but the dynamically generated 
bars don't. This 
behaviour refers to the native renderer (tested in Opera 9.6.4, 10.00, 10.10b1 
- OSX 10.5.8)

Once I set Opera to identify itself as Firefox the above version of the 
population pyramid draws as 
expected (you might have to quit Opera in between, probably for caching issues).

This issue hasn't always been present. The same population pyramid works fine 
in Opera
with a SVG Web release from around mid-September (r853?), see here
http://vis.uell.net/btw/09/poppyr11a.html

For some reasons the Flash renderer doesn't work on both of the above versions, 
no matter what 
(even identifying as Internet Exlorer doesn't help), but that's a minor issue.

Original issue reported on code.google.com by mneu...@gmail.com on 22 Oct 2009 at 6:00

GoogleCodeExporter commented 8 years ago
May I add that Opera has always been a very standard complient and fast 
implementation of SVG and although 
their market share is small we should definitely make sure that we don't run 
into issues with Opera.

In this respect the decission to lower the priority of Issue #126 "QA on Opera" 
might be reconsidered.
http://code.google.com/p/svgweb/issues/detail?id=126

Original comment by mneu...@gmail.com on 22 Oct 2009 at 6:09

GoogleCodeExporter commented 8 years ago
The problem is that Opera does not like the overrides in 
_patchCurrentTranslate()
that were introduced in r877.

Original comment by grick23@gmail.com on 23 Oct 2009 at 1:25

GoogleCodeExporter commented 8 years ago

Original comment by bradneub...@gmail.com on 13 Nov 2009 at 7:57

GoogleCodeExporter commented 8 years ago
I'll see if Opera can be patched in an alternative way for 
patchCurrentTranslate.

Original comment by bradneub...@gmail.com on 13 Nov 2009 at 7:58

GoogleCodeExporter commented 8 years ago
This is a blocker to get another release out.

Original comment by bradneub...@gmail.com on 19 Nov 2009 at 9:47

GoogleCodeExporter commented 8 years ago
I found a workaround that prevents this error from being thrown; note, though, 
that
the unit tests in general currently fail in Opera in both native and flash mode
(Issue 126). This means that while the workaround prevents an exception from 
being
thrown, it's unclear to me whether our currentTranslate patches work correctly 
on
Opera in general. Running correctly in Opera will have to await resolving Issue 
126.
At least this won't throw an error and be a blocker for Michael's Opera clients 
though.

Original comment by bradneub...@gmail.com on 19 Nov 2009 at 11:25

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r997.

Original comment by bradneub...@gmail.com on 19 Nov 2009 at 11:28