Thanks for the heads-up, but how does the empty reply detection work? It doesn't seem to do anything if I remove the patch. I click reply, click update ticket, and my 'empty reply' is recorded. Is it an option I have to turn on?<br>
<br>- Adam<br><br><div class="gmail_quote">On Wed, Jan 28, 2009 at 10:58 PM, Jesse Vincent <span dir="ltr"><<a href="mailto:jesse@bestpractical.com">jesse@bestpractical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Be careful. This patch will break rt's empty reply detection.<br>
<div><div></div><div class="Wj3C7c"><br>
Adam Tingle <<a href="mailto:adamtingle@gmail.com">adamtingle@gmail.com</a>> wrote:<br>
<br>
>Hi rt-users,<br>
><br>
>Here's a quick patch I made to add some whitespace to the top of the message<br>
>when replying to a message. It makes it easier to just start typing if you<br>
>have FCKeditor set to StartupFocus.<br>
><br>
><br>
>--- MessageBox  2009-01-28 21:37:01.000000000 -0500<br>
>+++ /usr/local/share/request-tracker3.8/html/Elements/MessageBox<br>
>2009-01-28 22:28:23.000000000 -0500<br>
>@@ -47,7 +47,7 @@<br>
> %# END BPS TAGGED BLOCK }}}<br>
> <textarea class="messagebox" cols="<% $Width %>" rows="<% $Height %>"<br>
>wrap="<% $Wrap %>" name="<% $Name %>">\<br>
> % $m->callback( %ARGS );<br>
>-<% $Default || '' %><% $message %><% $signature %></textarea><br>
>+<% $Default || '' %><% $whitespace %><% $message %><% $signature<br>
>%></textarea><br>
> <%INIT><br>
><br>
> my $message = '';<br>
>@@ -63,6 +63,10 @@<br>
>        $signature = "-- \n". $text;<br>
> }<br>
><br>
>+my $whitespace = '';<br>
>+if ( $message ) {<br>
>+    $whitespace = "\n\n\n";<br>
>+}<br>
> </%INIT><br>
> <%ARGS><br>
> $QuoteTransaction => undef<br>
><br>
><br>
>BTW, I caught on that making changes in<br>
>local/html/NoAuth/RichText/FCKeditor/fckconfig.js doesn't work; where's the<br>
>best place to override the FCKeditor configuration to enable the option<br>
>FCKConfig.StartupFocus  = true; ? Even better, how about making this the<br>
>default option, as no other fields seem to get the focus anyway.<br>
><br>
>--<br>
>...................<br>
>Adam Tingle<br>
</div></div>>_______________________________________________<br>
><a href="http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users" target="_blank">http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users</a><br>
><br>
>Community help: <a href="http://wiki.bestpractical.com" target="_blank">http://wiki.bestpractical.com</a><br>
>Commercial support: <a href="mailto:sales@bestpractical.com">sales@bestpractical.com</a><br>
><br>
><br>
>Discover RT's hidden secrets with RT Essentials from O'Reilly Media.<br>
>Buy a copy at <a href="http://rtbook.bestpractical.com" target="_blank">http://rtbook.bestpractical.com</a><br>
<font color="#888888">--<br>
Sent from my Android phone with K-9. Please excuse my brevity.</font></blockquote></div><br><br clear="all"><br>-- <br>...................<br>Adam Tingle<br><br>