Hi Jyrki,
script_at_top was indeed causing the error in the log files.
As for including this the "Application controller" - this is
undesired. The desired default is for the script to be loaded at the
bottom.
I thought the log output was going to be ok (and was in some tests),
because I was explicitly calling "flush" after closing off a block of
HTML. Apparently, I was wrong :)
script_at_top=true is desired and works well for commands like push/
pull. It would've been nice for log file outputting to allow
expansion of log commits before all logs finish rendering.
This issue has been fixed. Update to the latest.
Thanks!
Tim
On Mar 17, 1:29 pm, "Jyrki J." <jyrkij.li...@gmail.com> wrote:
> Hello,
> and thanks for the great plugin!
> I've been using git-tmbundle (as well as git) for a while only and am
> facing TextMate's rendering bug. When displaying logs the commits end
> up displayed in the wrong order.
> Why is the variable @script_at_top set to true everywhere? Wouldn't it
> be easier to add a before_filter to application controller rather than
> to every(?) controller? Anyhow, shouldn't @script_at_top be false
> because it's "causing" the rendering bug?
> Then there is this syntax error:
> <code>
> diff --git a/Support/app/views/log/layout.html.erb b/Support/app/views/
> log/layout.html.erb
> index 1ff6517..67a44b8 100644
> --- a/Support/app/views/log/layout.html.erb
> +++ b/Support/app/views/log/layout.html.erb
> @@ -15,5 +15,5 @@
> </body>
> <% unless @script_at_top %>
> <%= javascript_include_tag "prototype", "rb_gateway",
> "log_collapse" %>
> -<%= end %>
> +<% end %>
> </html>
> \ No newline at end of file
> </code>
> -jyrkij