Make the "this" variable point to the clicked link, not to the entire
window, so that it doesn't swallow up all other clicked events (and so that
clicking other links will still work).
Patch by fu.
diffs (12 lines):
diff -r b4d9f3a14173 -r 6b78ac69e1ce cgi-bin/weblib.pl
--- a/cgi-bin/weblib.pl Thu Apr 21 10:13:47 2011 +0800
+++ b/cgi-bin/weblib.pl Thu Apr 21 19:52:01 2011 +0800
@@ -1058,7 +1058,7 @@ sub make_qr_link
$basesubject =~ s/^(Re:\s*)*//i;
$basesubject = "Re: $basesubject" if $basesubject;
$basesubject = LJ::ehtml(LJ::ejs($basesubject));
- my $onclick = "return quickreply(\"$dtid\", $pid, \"$basesubject\")";
+ my $onclick = "return function(that) { return quickreply(\"$dtid\",
$pid, \"$basesubject\", that)}(this)";
my $r = DW::Request->get;
my $ju;
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog
