Skip to content

Commit 7983887

Browse files
committed
Toggle sponsor status based on data attributes on the tags, rather than the display text. Fixes #255
1 parent c504088 commit 7983887

1 file changed

Lines changed: 8 additions & 6 deletions

File tree

get_together/templates/get_together/events/manage_event_sponsors.html

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,9 @@ <h2>{% blocktrans with event_url=event.get_absolute_url event_name=event.name %}
5454
<strong>{{sponsor.name}}</strong><br/>
5555
<div class="btn-group">
5656
{% if sponsor in event.sponsors.all %}
57-
<a id="sponsor-button-{{sponsor.id}}" class="btn btn-danger" href="javascript:toggle_sponsor({{sponsor.id}});">{% trans "Remove" %}</a></span>
57+
<a id="sponsor-button-{{sponsor.id}}" data-action="remove" class="btn btn-danger" href="javascript:toggle_sponsor({{sponsor.id}});">{% trans "Remove" %}</a></span>
5858
{% else %}
59-
<a id="sponsor-button-{{sponsor.id}}" class="btn btn-success" href="javascript:toggle_sponsor({{sponsor.id}});">{% trans "Add" %}</a></span>
59+
<a id="sponsor-button-{{sponsor.id}}" data-action="add" class="btn btn-success" href="javascript:toggle_sponsor({{sponsor.id}});">{% trans "Add" %}</a></span>
6060
{% endif %}
6161
</div>
6262
</div>
@@ -96,19 +96,21 @@ <h2>{% blocktrans with event_url=event.get_absolute_url event_name=event.name %}
9696
function toggle_sponsor(sponsor_id) {
9797
var btn = $("#sponsor-button-"+sponsor_id)
9898
var value = ''
99-
if (btn[0].innerText == 'Add') {
99+
if (btn[0].dataset.action == 'add') {
100100
value = 'add'
101-
} else if (btn[0].innerText == 'Remove') {
101+
} else if (btn[0].dataset.action == 'remove') {
102102
value = 'remove'
103103
}
104104
$.getJSON("{% url 'sponsor-event' event.id %}?sponsor="+sponsor_id+"&action="+value, function(data, status) {
105105
console.log(data)
106106
if (data.status == "OK") {
107107
if (data.action == "Added") {
108-
btn[0].innerText = "Remove"
108+
btn[0].innerText = "{% trans 'Remove' %}"
109+
btn[0].dataset.action = 'remove'
109110
btn.removeClass('btn-success').addClass('btn-danger')
110111
} else if (data.action == "Removed") {
111-
btn[0].innerText = "Add"
112+
btn[0].innerText = "{% trans 'Add' %}"
113+
btn[0].dataset.action = 'add'
112114
btn.removeClass('btn-danger').addClass('btn-success')
113115
}
114116
}

0 commit comments

Comments
 (0)