I have sent a new patch solving the issues you point.<br><br>Gonzalo<br><br><div class="gmail_quote">On Mon, Jan 9, 2012 at 10:24 AM, Aleksey Lim <span dir="ltr"><<a href="mailto:alsroot@activitycentral.org">alsroot@activitycentral.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Jan 05, 2012 at 10:47:33AM -0300, Gonzalo Odiard wrote:<br>
</div><div class="im">> Hi Aleksey,<br>
> I have addressed the pending issues, and will send the patches to<br>
> sugar-devel.<br>
<br>
</div>Thanks, I've pushed your commits w/ lint polishing (see HACKING file).<br>
<a href="http://git.sugarlabs.org/chat/mainline/commit/148d0d6fd47d8eb6b3bf42d21828222ee4693082" target="_blank">http://git.sugarlabs.org/chat/mainline/commit/148d0d6fd47d8eb6b3bf42d21828222ee4693082</a><br>
<br>
Also, there are a couple of issues:<br>
<br>
    diff --git a/chat/box.py b/chat/box.py<br>
    index bfc8156..b639037 100644<br>
    --- a/chat/box.py<br>
    +++ b/chat/box.py<br>
    @@ -69,9 +69,11 @@ class TextBox(gtk.TextView):<br>
             self._empty = True<br>
             self.palette = None<br>
             self._mouse_detector = MouseSpeedDetector(self, 200, 5)<br>
    -        self._mouse_detector.connect('motion-slow', self._mouse_slow_cb)<br>
    +        # TODO There is a mess with having sugar palette and TextView popup<br>
    +        #self._mouse_detector.connect('motion-slow', self._mouse_slow_cb)<br>
             self.modify_base(gtk.STATE_NORMAL, bg_color.get_gdk_color())<br>
    -        self.connect("event-after", self.event_after)<br>
    +        # TODO There is a mess with having sugar palette and TextView popup<br>
    +        #self.connect("event-after", self.event_after)<br>
             self.motion_notify_id = self.connect("motion-notify-event", \<br>
                     self.motion_notify_event)<br>
             self.connect("visibility-notify-event", self.visibility_notify_event)<br>
<br>
For example,<br>
<br>
* in self.event_after, the commented block references to unknow `tag`<br>
* having two popups (sugar palette and TextBox popup) looks messy.<br>
* url palette pops up even for TextBox widgets that don't have urls<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Aleksey<br>
</font></span></blockquote></div><br>