[Jifty-commit] r5306 - jifty/branches/jquery/lib/Jifty/Plugin

Jifty commits jifty-commit at lists.jifty.org
Tue Apr 15 11:25:20 EDT 2008


Author: gugod
Date: Tue Apr 15 11:25:18 2008
New Revision: 5306

Modified:
   jifty/branches/jquery/lib/Jifty/Plugin/Halo.pm

Log:
Fix a typo where Jifty.$() is used an qq string and $( is really a
variable there.  Need to escape that $ sign.


Modified: jifty/branches/jquery/lib/Jifty/Plugin/Halo.pm
==============================================================================
--- jifty/branches/jquery/lib/Jifty/Plugin/Halo.pm	(original)
+++ jifty/branches/jquery/lib/Jifty/Plugin/Halo.pm	Tue Apr 15 11:25:18 2008
@@ -197,8 +197,7 @@
                 if ($ref) {
                     my $expanded = Jifty->web->serial;
                     my $yaml = Jifty->web->escape(Jifty::YAML::Dump($value));
-                    #$out .= qq{<a href="#" onclick="Element.toggle('$expanded'); return false">$ref</a><div id="$expanded" style="display: none; position: absolute; left: 200px; border: 1px solid black; background: #ccc; padding: 1em; padding-top: 0; width: 300px; height: 500px; overflow: auto"><pre>$yaml</pre></div>};
-                    $out .= qq{<a href="#" onclick="jQuery(Jifty.$('$expanded')).toggle()); return false">$ref</a><div id="$expanded" class="halo-argument" style="display: none"><pre>$yaml</pre></div>};
+                    $out .= qq{<a href="#" onclick="jQuery(Jifty.\$('$expanded')).toggle()); return false">$ref</a><div id="$expanded" class="halo-argument" style="display: none"><pre>$yaml</pre></div>};
                 }
                 elsif (defined $value) {
                     $out .= Jifty->web->escape($value);


More information about the Jifty-commit mailing list