I'm unconvinced that it will be a short term workaround. It will be hard to get rid of.<div><br></div><div>We are implicitly adding support for webkit 1.8 (API 1) to sugar-web. We might have to write sugar-web code which is webkit1 specific. People will write activities which runs on 1.8. They will use non forward-compatible APIs. They will cut and paste python wrappers to run on webkit1. For a long time webkit <= 1.8 will be the main platform really, given the deployments situation.</div>
<div><br></div><div>I think this is an important decision. Even if it's not our intention we will either send the message that we would like people to start using the web toolkit on deployed software or that it's still a cutting edge technology, targeted to developers and very recent software. The are advantages and disadvantages of both approaches, the point is more that we should be aware of the choice we are making.</div>
<div><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">I am not opposed to alternative implementations,<div>
if we can "keep it simple".</div><div><br></div><div>This is a workaround for a problem, in the end, we want remove it,</div><div>when is not needed anymore.</div>
<div><br></div><div>Gonzalo</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 27, 2013 at 2:22 PM, Rogelio Mita <span dir="ltr"><<a href="javascript:_e({}, 'cvml', 'rogeliomita@activitycentral.com');" target="_blank">rogeliomita@activitycentral.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Nice!, I quickly looked at the patch and I urged try to encapsulate the all logic of webkit2 and the webkit1 in separate modules, using a luck of strategy pattern or do something polymorphic, but I notice that it is a decision not yet taken (as Daniel says), I doubt whether to spend time on this for that reason, you will say, however when this decision is finally decided, I would like to make note of this style refactor to have clean webactivity module, which is relatively new and keep small, you think?<br>
<div><br></div><div>Regards!</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/27 Gonzalo Odiard <span dir="ltr"><<a href="javascript:_e({}, 'cvml', 'gonzalo@laptop.org');" target="_blank">gonzalo@laptop.org</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><div>On Wed, Nov 27, 2013 at 1:04 PM, Daniel Narvaez <span dir="ltr"><<a href="javascript:_e({}, 'cvml', 'dwnarvaez@gmail.com');" target="_blank">dwnarvaez@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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></blockquote><div><br></div></div><div>Maybe, but have many other dependencies to update.</div><div><br></div><div>Another alternative is find what is crashing in webkit2 and solve it, but is out of my knowledge.</div>
<span><font color="#888888">
<div><br></div><div>Gonzalo</div><div><br></div></font></span></div></div></div>
<br></div></div>_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="javascript:_e({}, 'cvml', 'Sugar-devel@lists.sugarlabs.org');" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
<br></blockquote></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br>Roger<div><br><div><a href="http://activitycentral.com/" target="_blank">Activity Central</a></div></div>
</font></span></div>
</blockquote></div><br></div>
</blockquote></div><br><br>-- <br>Daniel Narvaez<br><br>