<div dir="ltr">cc: manuq, maintainer of Browse</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 18, 2013 at 4:37 PM, Walter Bender <span dir="ltr"><<a href="mailto:walter.bender@gmail.com" target="_blank">walter.bender@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Nov 18, 2013 at 2:23 PM, Sai Vineet <<a href="mailto:saivineet89@gmail.com">saivineet89@gmail.com</a>> 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<br>
> this review may take time, so I should move on the other tasks after<br>
> unclaiming this task.<br>
><br>
> Please review this fast. Thank you!<br>
><br>
</div></div>> _______________________________________________<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>
I've tested the patch: it works. The question is whether or not we<br>
like the behavior. The relevant ticket is<br>
<a href="http://bugs.sugarlabs.org/ticket/3954" target="_blank">http://bugs.sugarlabs.org/ticket/3954</a><br>
What happens is that escape will both close the suggestions drop-down<br>
and clear the entry. It may be better to do it in two steps.<br>
<span class="HOEnZb"><font color="#888888"><br>
-walter<br>
<br>
--<br>
Walter Bender<br>
Sugar Labs<br>
<a href="http://www.sugarlabs.org" target="_blank">http://www.sugarlabs.org</a><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>
</font></span></blockquote></div><br></div>