Gary,<div><br></div><div>Looking forward to your feedback.</div><div><br></div><div>Regards,</div><div><br></div><div>Manu<br><br><div class="gmail_quote">On Tue, Nov 30, 2010 at 2:43 PM, Sascha Silbe <span dir="ltr"><<a href="mailto:sascha-ml-reply-to-2010-3@silbe.org">sascha-ml-reply-to-2010-3@silbe.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Excerpts from anubhav's message of Tue Nov 30 06:49:42 +0100 2010:<br>
<br>
You break threading by supplying an invalid message id in the following<br>
headers:<br>
<br>
> In-Reply-To: <no><br>
> References: <no><br>
<br>
This makes it rather hard for me to review your patch. If you'd like me<br>
to review your patch, please submit it again without passing<br>
"--in-reply-to=no" to git send-email.<br>
<br>
You can also adjust the subject. The patch isn't a feature request, it's<br>
an implementation of a feature. A better summary of what your patch does<br>
would be:<br>
<br>
Add button in activity toolbar to invoke the Name Alert<br>
<br>
<br>
I'm also not exactly fond of the Name Alert. I'd much rather have that<br>
button invoke the Journal details view instead. But that's my personal<br>
opinion, so I recommend you wait until the Design Team has agreed on a<br>
choice.<br>
<br>
Sascha<br>
<font color="#888888"><br>
--<br>
<a href="http://sascha.silbe.org/" target="_blank">http://sascha.silbe.org/</a><br>
<a href="http://www.infra-silbe.de/" target="_blank">http://www.infra-silbe.de/</a><br>
</font><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></blockquote></div><br></div>