dw-free: http://bugs.dwscoalition.org/show_bug.cgi?id=3634

[Dreamwidth Changelog]

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;