<div dir="ltr">On 6 November 2013 18:32, Gonzalo Odiard <span dir="ltr"><<a href="mailto:gonzalo@laptop.org" target="_blank">gonzalo@laptop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="im">>> No. the web-server issue is already solved.<br>
><br>
> If we want to support WebKit1, I think we should do it upstream then. I'm<br>
> still not thrilled about that but not opposed to it either.<br>
<br>
</div>Good. We can see what is the better way to do it. May be check the<br>
webkit version installed?<br>
<br>
Just for reference, the changes we did are here:<br>
<br>
<a href="https://github.com/godiard/au1b_rpms/blob/master/sugar-toolkit-gtk3/0001-Adapt-webactivity-to-work-with-webkit1.patch" target="_blank">https://github.com/godiard/au1b_rpms/blob/master/sugar-toolkit-gtk3/0001-Adapt-webactivity-to-work-with-webkit1.patch</a><br>

<span class=""></span></blockquote><div><br></div><div>I think the best way to detect it would be to import WebKit2, if it fails fall back to WebKit.<br><br>I'm not sure what's the best way to maintain the two different implementations though. Maybe two modules with identical interfaces, imported "as webview".<br>
<br>So something like this in webactivity.py:<br><br></div><div>try:<br></div><div>  from gi.repository import WebKit2<br></div><div>except ImportError:<br></div><div>  pass <br></div><div><br>if WebKit2:<br></div><div>  from sugar.activity import webkit1view as webview<br>
else:<br></div><div>  from sugar.activity import webkit2view as webview<br></div><div><br></div><div>class WebActivity:<br></div><div>  def __init__(self):<br>    self._web_view = webview.WebView()<br>...<br></div></div></div>
</div>