Le mercredi 14 octobre 2009 à 08:34 +0200, Julien Valroff a écrit :
> Hi,
> 
> Le dimanche 16 août 2009 à 17:19 +0000, The Fungi a écrit :
> > Package: ajaxterm
> > Version: 0.10-4
> > Severity: wishlist
> > Tags: l10n patch
> > 
> > Please consider implementing the patch written by Sergej Pupykin
> > <[email protected]> to switch ajaxterm from Latin-1 to UTF-8 encoding, as
> > this is the default encoding for Debian's upcoming release. I have
> > attached a tweaked and rebased patch which applies cleanly from the
> > series in the debian/patches directory of the current source
> > package. I have extensively tested this and found it to work well.
> 
> I have an issue with your path on IE8 (not tested on earlier version).
> 
> The following lines in ajaxterm.js lead to an error "Object required":
> 
> de=r.responseXML.documentElement;
> if (de.tagName == "pre") {
> 
> The corresponding part of the ajaxterm.py is:
> span+=u'\n'
> r='<?xml version="1.0" encoding="utf8"?><pre class="term">%s</pre>'%r

My bad, that was the part we had removed previously which is actually required!
-                                               
de=r.responseXML.documentElement;
+                                               if(ie)
+                                               {
+                                                   var responseXMLdoc = new 
ActiveXObject("Microsoft.XMLDOM");
+                                                   
responseXMLdoc.loadXML(r.responseText);
+                                                   de = 
responseXMLdoc.documentElement;
+                                               }
+                                               else
+                                               {
+                                                   
de=r.responseXML.documentElement;
+                                               }

Now, still some issues on ie8 on Windows Seven: the console is not of
the expected size (does only adapt to the contents displayed) and the
red cursor is one line after the prompt, and is much longer than it
should be).
It does not happen without the patch.

Would you please have a look at it?

Cheers,
Julien






--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to