<br><br><div class="gmail_quote">On Wed, Nov 3, 2010 at 4:09 PM, Anish Mangal <span dir="ltr"><<a href="mailto:anishmangal2002@gmail.com">anishmangal2002@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi,<br>
<br>
My two cents on this...<br>
<div class="im"><br>
0001-S-non-intrusive-NamingAlert.patch<br>
<br>
</div>+        <long>If TRUE NamingAlert will show when<br>
+        the activity close.</long><br>
<br>
s/If TRUE/If TRUE,/  (comma)<br>
s/show/popup/<br>
s/close/closes/<br>
<br>
***<br>
<div class="im"><br>
0001-STK-non-intrusive-NamingAlert.patch<br>
<br>
</div>-        if skip_save or self._jobject is None or \<br>
+        client = gconf.client_get_default()<br>
+        show_alert = client.get_bool('/desktop/sugar/show_naming_alert')<br>
<br>
/me being nitpicky, but perhaps show_naming_alert would be a better<br>
variable name as compared so show_alert<br></blockquote><div><br>you are right, but i thought it was a little too long and did not want to break<br>the line just because of that. I think the identifier self explains enough in this<br>
case :).<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Also, don't know if the commit title message could be better. "Add<br>
show_naming_alert gconf key" or something like that instead of "S<br>
non-intrusive NamingAlert"<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div><br>Right again, i should mention the key name at least on the comments.<br><br>Thanks a lot for the review!<br><br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div class="h5">
On Wed, Nov 3, 2010 at 4:06 AM, Martin Abente<br>
<<a href="mailto:martin.abente.lahaye@gmail.com">martin.abente.lahaye@gmail.com</a>> wrote:<br>
> Hello amigos,<br>
><br>
> I keep following the dextrose TODO list [1]. This time I made simple patches<br>
> to make NamingAlert (the details popup that appears when the activity<br>
> closes) optional.<br>
> These patches should cleanly apply to Dextrose code base.<br>
><br>
> 0001-S-non-intrusive-NamingAlert.patch applies to sugar.<br>
> 0001-STK-non-intrusive-NamingAlert.patch applies to sugar-toolkit<br>
><br>
> This is a review request :)<br>
><br>
> 1 .<a href="http://wiki.sugarlabs.org/go/Dextrose/TODO" target="_blank">http://wiki.sugarlabs.org/go/Dextrose/TODO</a><br>
><br>
</div></div>> _______________________________________________<br>
> Dextrose mailing list<br>
> <a href="mailto:Dextrose@lists.sugarlabs.org">Dextrose@lists.sugarlabs.org</a><br>
> <a href="http://lists.sugarlabs.org/listinfo/dextrose" target="_blank">http://lists.sugarlabs.org/listinfo/dextrose</a><br>
><br>
><br>
<br>
Cheers,<br>
<font color="#888888">Anish<br>
</font></blockquote></div><br>