Merge lp:~michael.nelson/ubuntu-webcatalog/1039073-click-url-should-open-in-own-window into lp:ubuntu-webcatalog

Proposed by Michael Nelson
Status: Merged
Approved by: Łukasz Czyżykowski
Approved revision: 165
Merged at revision: 164
Proposed branch: lp:~michael.nelson/ubuntu-webcatalog/1039073-click-url-should-open-in-own-window
Merge into: lp:ubuntu-webcatalog
Diff against target: 12 lines (+1/-1)
1 file modified
src/webcatalog/templates/webcatalog/exhibits_widget.html (+1/-1)
To merge this branch: bzr merge lp:~michael.nelson/ubuntu-webcatalog/1039073-click-url-should-open-in-own-window
Reviewer Review Type Date Requested Status
Łukasz Czyżykowski (community) Approve
Review via email: mp+120561@code.launchpad.net

Description of the change

Tiny change so that clicks on an exhibit open the url in a new tab.

At first I was unsure if we should do this for all exhibits, or just those with click_url's (ie. external), but it seemed inconsistent to do it only for some exhibits. That said, if it's related to the external/internal link, maybe it would be better only do on external? Let me know.

I'm personally -1 to opening in a new tab/window - I don't really see the benefit in our use-case.

To post a comment you must log in.
Revision history for this message
Łukasz Czyżykowski (lukasz-czyzykowski) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/webcatalog/templates/webcatalog/exhibits_widget.html'
2--- src/webcatalog/templates/webcatalog/exhibits_widget.html 2012-03-08 18:00:09 +0000
3+++ src/webcatalog/templates/webcatalog/exhibits_widget.html 2012-08-21 13:49:19 +0000
4@@ -10,7 +10,7 @@
5 {% endcomment %}
6 {% for exhibit in exhibits %}
7 <li class="slide{% if forloop.counter > 1%} disabled{% endif %}">
8- <a href="{{ exhibit.destination_url }}">
9+ <a {% if exhibit.click_url %}target="_blank"{% endif %} href="{{ exhibit.destination_url }}">
10 {{ exhibit.html }}
11 </a>
12 </li>

Subscribers

People subscribed via source and target branches