Alec Thomas wrote:
> I'm migrating Trac Hacks to 0.11 and for some reason the in-place
> browser expansion isn't working:
>
> http://staging.trac-hacks.org/browser
>
> Any ideas what's going on?
>
>
Looks like there's an empty line before the <DOCTYPE> in the result XHR
that messes up the Javascript replace(/^<DOCTYPE...).
That's apparently related to your latest changeset in Genshi ([G787]).
The following change will fix it, but I wonder if that empty line
leading line is not a buglet in itself?
diff --git a/trac/htdocs/js/expand_dir.js b/trac/htdocs/js/expand_dir.js
--- a/trac/htdocs/js/expand_dir.js
+++ b/trac/htdocs/js/expand_dir.js
@@ -81,7 +81,7 @@ function toggleDir(expander, qargs) {
data: qargs,
dataType: "html",
success: function(data) {
- var rows = $(data.replace(/^<!DOCTYPE[^>]+>/, "")).filter("tr");
+ var rows = $(data.replace(/^\s*<!DOCTYPE[^>]+>/, "")).filter("tr");
if (rows.length) {
// insert entry rows
rows.children("td."+td_class).css("padding-left", depth);
-- Christian
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Trac
Development" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/trac-dev?hl=en
-~----------~----~----~----~------~----~------~--~---
