From ba55a6e68f32184ff897f20681fcbc96d0225825 Mon Sep 17 00:00:00 2001
From: Max Goodman <max@reddit.com>
Date: Fri, 15 Nov 2013 03:18:24 -0700
Subject: [PATCH] Parameterize SideBox link target attribute.

---
 r2/r2/lib/pages/pages.py     | 4 ++--
 r2/r2/templates/sidebox.html | 3 ++-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/r2/r2/lib/pages/pages.py b/r2/r2/lib/pages/pages.py
index db7d0df3a..74e9d35e3 100644
--- a/r2/r2/lib/pages/pages.py
+++ b/r2/r2/lib/pages/pages.py
@@ -833,8 +833,8 @@ class SideBox(CachedTemplate):
     """
     def __init__(self, title, link=None, css_class='', subtitles = [],
                  show_cover = False, nocname=False, sr_path = False,
-                 disabled=False, show_icon=True):
-        CachedTemplate.__init__(self, link = link, target = '_top',
+                 disabled=False, show_icon=True, target='_top'):
+        CachedTemplate.__init__(self, link = link, target = target,
                            title = title, css_class = css_class,
                            sr_path = sr_path, subtitles = subtitles,
                            show_cover = show_cover, nocname=nocname,
diff --git a/r2/r2/templates/sidebox.html b/r2/r2/templates/sidebox.html
index 92b83e4f6..b6cdb621d 100644
--- a/r2/r2/templates/sidebox.html
+++ b/r2/r2/templates/sidebox.html
@@ -25,7 +25,8 @@
 <div class="sidebox ${thing.css_class}${' disabled' if thing.disabled else ''}">
   <div class="morelink">
     ${plain_link(thing.title, thing.link, _sr_path=thing.sr_path,
-                 _class='login-required' if thing.show_cover else None, nocname=thing.nocname)}
+                 _class='login-required' if thing.show_cover else None, nocname=thing.nocname,
+                 target=thing.target)}
     <div class="nub"> </div>
   </div>