Summary: Fix bugs in implementation of add/remove from community list JS
- Legacy-Id: 8967
This commit is contained in:
parent
a5aa58e0b2
commit
703d9995ea
|
@ -338,9 +338,9 @@
|
|||
{% endif %}
|
||||
{% if user.is_authenticated %}
|
||||
{% if tracking_document %}
|
||||
<a class="btn btn-default btn-xs" href="{% url "community_remove_track_document" doc.name %}" title="Remove from your personal ID list"><span class="fa fa-bookmark-o"></span> Untrack</a>
|
||||
<a class="btn btn-default btn-xs community-list-add-remove-doc" href="{% url "community_remove_track_document" doc.name %}" title="Remove from your personal ID list"><span class="fa fa-bookmark-o"></span> Untrack</a>
|
||||
{% else %}
|
||||
<a class="btn btn-default btn-xs" href="{% url "community_add_track_document" doc.name %}" title="Add to your personal ID list"><span class="fa fa-bookmark"></span> Track</a>
|
||||
<a class="btn btn-default btn-xs community-list-add-remove-doc" href="{% url "community_add_track_document" doc.name %}" title="Add to your personal ID list"><span class="fa fa-bookmark"></span> Track</a>
|
||||
{% endif %}
|
||||
{% endif %}
|
||||
|
||||
|
|
|
@ -6,12 +6,12 @@
|
|||
<td>
|
||||
{% if user.is_authenticated %}
|
||||
{% if not doc.name in have_doc_status %}
|
||||
{% elif doc.name in doc_is_tracked %}
|
||||
<a href="{% url "community_remove_track_document" doc.name %}" title="Remove from your personal ID list">
|
||||
<span class="fa fa-bookmark-o"></span>
|
||||
{% elif doc.name in doc_is_tracked %}
|
||||
<a href="{% url "community_remove_track_document" doc.name %}" class="community-list-add-remove-doc" title="Remove from your personal ID list">
|
||||
<span class="fa fa-bookmark-o"></span>
|
||||
</a>
|
||||
{% else %}
|
||||
<a href="{% url "community_add_track_document" doc.name %}" title="Add to your personal ID list">
|
||||
<a href="{% url "community_add_track_document" doc.name %}" class="community-list-add-remove-doc" title="Add to your personal ID list">
|
||||
<span class="fa fa-bookmark"></span>
|
||||
</a>
|
||||
{% endif %}
|
||||
|
|
|
@ -100,7 +100,7 @@ $(document).ready(function () {
|
|||
}
|
||||
|
||||
// search results
|
||||
$('.addtolist a').click(function(e) {
|
||||
$('.community-list-add-remove-doc').click(function(e) {
|
||||
e.preventDefault();
|
||||
var trigger = $(this);
|
||||
$.ajax({
|
||||
|
@ -110,8 +110,10 @@ $(document).ready(function () {
|
|||
dataType: 'json',
|
||||
success: function(response){
|
||||
if (response.success) {
|
||||
trigger.replaceWith('<span class="fa fa-tag text-danger"></span>');
|
||||
}
|
||||
trigger.parent().find(".tooltip").remove();
|
||||
// it would be neater to swap in remove link
|
||||
trigger.replaceWith('<span class="fa fa-tag text-danger"></span>');
|
||||
}
|
||||
}
|
||||
});
|
||||
});
|
||||
|
|
Loading…
Reference in a new issue