<div dir="ltr">Here's the modified patch as per James Cameron's requirements.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 19, 2013 at 3:24 AM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Tested-by: James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>><br>
<br>
- it works for me, discarding both the URL text, hiding the X delete<br>
  icon, and hiding the history search results.<br>
<br>
- the single key press solution is acceptable to me, having to press<br>
  escape twice would be irritating,<br>
<br>
Reviewed-by: James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>><br>
<br>
- the patch narrative might have more text in subsequent lines<br>
  describing the problem and how it is solved, with a reference to the<br>
  Sugar Labs ticket number.<br>
<div><div class="h5"><br>
On Tue, Nov 19, 2013 at 12:53:12AM +0530, Sai Vineet wrote:<br>
> I've attached the patch. It is a one liner.<br>
> The mentors in GCI, namely Walter Bender and Gonzalo Odiard, told me that this<br>
> review may take time, so I should move on the other tasks after unclaiming this<br>
> task.<br>
><br>
> Please review this fast. Thank you!<br>
<br>
</div></div>> From 79a02c411594eacf4af2d796e6c1acfb7a241317 Mon Sep 17 00:00:00 2001<br>
> From: Sai Vineet <<a href="mailto:saivineet89@gmail.com">saivineet89@gmail.com</a>><br>
> Date: Mon, 18 Nov 2013 23:56:40 +0530<br>
> Subject: [PATCH 1/1] Added clear URL bar with Escape key feature<br>
><br>
> ---<br>
>  webtoolbar.py |    1 +<br>
>  1 file changed, 1 insertion(+)<br>
><br>
> diff --git a/webtoolbar.py b/webtoolbar.py<br>
> index b6ad558..2fb98f5 100644<br>
> --- a/webtoolbar.py<br>
> +++ b/webtoolbar.py<br>
> @@ -231,6 +231,7 @@ class WebEntry(iconentry.IconEntry):<br>
>              return True<br>
>          elif keyname == 'Escape':<br>
>              self._search_window.hide()<br>
> +            self.props.text = ''<br>
>              return True<br>
>          return False<br>
><br>
> --<br>
> 1.7.9.5<br>
<div class="HOEnZb"><div class="h5">><br>
<br>
> _______________________________________________<br>
> Sugar-devel mailing list<br>
> <a href="mailto:Sugar-devel@lists.sugarlabs.org">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>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</font></span></blockquote></div><br></div>