I think it's good to have this upstream because otherwise changes to sugar-web will easily break webkit1 support.<div><br></div><div>I'm not completely convinced we should support webkit1 yet. I'm worried it will be pretty painful to maintain. It should be possible to port webkitgtk 2.2 to Fedora 18... </div>
<div><br></div><div>Though I suppose it could also be useful for some older OLPC releases with webkit, if any of those are or will be used in the field... And you are doing the work anyway, the approach is reasonable, I won't strongly object :)<br>
<br>On Wednesday, 27 November 2013, Gonzalo Odiard  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">When we started the work in web activities, we decided work with webkit2,<div>
because is possible solve a few problems in the client side,</div><div>without need a local webserver.</div><div><br></div><div>But right now, in the XOs, we have F18 working, and Webkit2 crashes,</div>
<div>then I didn't find other solution than modify webactivity to work with webkit1.</div><div>That code was included in the rpms I prepared and the testing images.</div><div><br></div><div>I would like propose include this change in master,</div>

<div>used only when a environment variable is set, to make easier the testing,</div><div>and be sure the changes we do works in both webkit1 and webkit2.</div><div><br></div><div>A first proposal is available here <a href="https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/54" target="_blank">https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/54</a></div>

<div><br></div><div>Thoughts? </div><div><br></div><div>Gonzalo</div></div>
</blockquote></div><br><br>-- <br>Daniel Narvaez<br><br>