Reputation: 2284
My application requires that users upload a set of required documents (Upload #1, Upload #2, Upload #3). When an upload completes, it returns the requirement identifier (req_id).
The page displays a list of the requirements, and currently changes the class from "missing" to "complete" once a particular item is uploaded. However, I'd also like to change the visibility of a "delete" icon from hidden to visible once the upload is complete as well.
HTML:
<ul>
<li class="missing" rel="1">
<span class="link">
<a href="#" target="_blank">Upload #1</a>
</span>
<span class="controls">
<img src="download.png" class="download" <? if ($req_item['class'] == "missing") { echo 'style="visibility: hidden;"'; } ?> >
<img src="trash.png" class="delete" <? if ($req_item['class'] == "missing") { echo 'style="visibility: hidden;"'; } ?> >
</span>
</li>
<li class="missing" rel="2">
<span class="link">
<a href="#" target="_blank">Upload #2</a>
</span>
<span class="controls">
<img src="download.png" class="download" <? if ($req_item['class'] == "missing") { echo 'style="visibility: hidden;"'; } ?> >
<img src="trash.png" class="delete" <? if ($req_item['class'] == "missing") { echo 'style="visibility: hidden;"'; } ?> >
</span>
</li>
<li class="missing" rel="3">
<span class="link">
<a href="#" target="_blank">Upload #3</a>
</span>
<span class="controls">
<img src="download.png" class="download" <? if ($req_item['class'] == "missing") { echo 'style="visibility: hidden;"'; } ?> >
<img src="trash.png" class="delete" <? if ($req_item['class'] == "missing") { echo 'style="visibility: hidden;"'; } ?> >
</span>
</li>
</ul>
Javascript:
function stopUpload(success, req_id){
if (success == 1){
$('#upload_result').html('<span class="msg">Success!<\/span>');
$("ul li[rel=" + req_id + "]").removeClass().addClass('complete')
$("ul li[rel=" + req_id + "]").child('.controls').child('.delete').css('visibility','visible')
}
else {
$('#upload_result', window.parent.document).html(
'<span class="emsg">Error!<\/span>');
}
$('#upload_progress').hide();
return true;
}
Upvotes: 3
Views: 21319
Reputation: 4415
To use jQuery, replace
$("ul li[rel=" + req_id + "]").removeClass().addClass('complete')
$("ul li[rel=" + req_id + "]").child('.controls').child('.delete').css('visibility','visible')
By:
$("ul li[rel=" + req_id + "]").removeClass().addClass('complete').find('.controls .delete').show();
However, you should use CSS for this. So keep your original jQuery and add this to CSS:
ul li .controls .delete {
display: block;
}
ul li.complete .controls .delete {
display: none;
}
Upvotes: 4
Reputation: 3829
You want to change
$("ul li[rel=" + req_id + "]").child('.controls').child('.delete').css('visibility','visible')
to
$("ul li[rel=" + req_id + "] .controls .delete").show()
Should work nicely.
Rather than visibility, I'd tend to use the display property - display:none;
or display:block;
or display:inline;
Upvotes: 0